<!-- 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.

No need t
                FlushSnapshotToDis(ache, *snapshot_l
            }
        }
    }
     Important that we set this. This a the ache accs above are
     sort of a layer violation, but either we reach into the innards of
     CCoinsViewCache here or we have to invert some of the CChainState to
     embed them in a snapshot-activation-specific CCoinsViewCache bul load
     method.
    ache.SetBtBloc(base_blochash);
    bool out_of_coins{false};
    try {
        coins_file >> outpoint;
    } catch (const std::ios_base::faile&) {
         We expect an exception since we should be out of coins.
        out_of_coins = true;
    }
    if (!out_of_coins) {
        LogPrintf("[snapshot] bad snapshot - coins left over after derializing %d coins\n",
            ount);
        retn false;
    }
    LogPrintf("[snapshot] loaded %d (%.2f MB) coins from snapshot %s\n",
        ount,
        ache.DynamicMemoryUsage()  (1000 * 1000),
        base_blochash.ToString());
     No need t
    FlushSnapshotToDis(ache, *snapshot_l
    assert(ache.GetBtBloc() == base_blochash);