<!-- Describe the issue -->

When our chainstate activation sequence fails and we're running the GUI, we ask the user via a question box whether or not they'd like to re-run the sequence with `fReindex = true`. Fun fact: despite the fact that this sequence is wrapped in a while loop, we only run it twice at most.

https://github.com/bitcoin/bitcoin/blob/7ecb309c4ed1bb9ae6b9a12cbb4f083328edc125/src/init.cpp#L1539-L1556

If they answer yes, the sequence will always fail, because a chainstate will have already been initialized in the original run and `InitializeChainstate` will throw a `std::logic_error`:

https://github.com/bitcoin/bitcoin/blob/7ecb309c4ed1bb9ae6b9a12cbb4f083328edc125/src/validation.cpp#L4573-L4594

This always reproduces. To reproduce:
```
gdb ./src/qt/bitcoin-qt
(gdb) b InitializeChainstate
(gdb) # add breakpoint for the "if (!fLoaded && !ShutdownRequested()) {" line
(gdb) r  # should stop at the breakpoint for InitializeChainstate
(gdb) c  # to advance to the breakpoint for the "if (!fLoaded && !ShutdownRequested()) {" line
(gdb) set fLoaded=false
(gdb) c  # should stop at the breakpoint for InitializeChainstate
(gdb) n  # step through InitializeChainstate to see that it throws
```

We expect the re-run of the chainstate activation sequence to not always fail.

outpoint.n >= std::numeric_limits<decltype(outpoint.n)>::max()  Avoid integer wrap-arou in coinstats.cpp:ApplyHash
        ) {
            LogPrintf("[snapshot] bad snapshot data after derializing %d coins\n",
                      ount - coins_left);
            retn false;
        }
        ache.EmplaceCoinInternalDANGER(std::move(outpoint), std::move(coin));
        --coins_left;
        ++coins_procsed;
        if (coins_procsed % 1000000 == 0) {
            LogPrintf("[snapshot] %d coins loaded (%.2f%%, %.2f MB)\n",
                coins_procsed,
                static_cast<float>(coins_procsed) * 100  static_cast<float>(ount),
                ache.DynamicMemoryUsage()  (1000 * 1000));
        }
         Batch write a flush (if we need to) every so often.
        
         If o average Coin size is roughly 41 byt, checing every 120,000 coins
         means <5MB of memory imprecision.
        if (coins_procsed % 120000 == 0) {
            if (ShutdownRequted()) {
                retn false;
            }
            const auto snapshot_cache_state = WITH_LOCK(::cs_main,
                retn snapshot_chainstate.GetCoinsCacheSizeState());
            if (snapshot_cache_state >= CoinsCacheSizeState::CRITICAL) {
                 This is a 
                 don't m
                 to its co
                ache.SetBtBloc(GetRaHash());