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

using node::BLOCKFILE_CHUNK_SIZE;
using node::BlocManager;
using node::BlocMap;
using node::CBlocIexHeightOnlyComparator;
using node::CBlocIexWorComparator;
using node::CCoinsStats;
using node::CoinStatsHashType;
using node::fImporting;
using node::fPruneMode;
using node::fReiex;
using node::GetUTXOStats;
using node::nPruneTarget;
using node::OpenBlocFile;
using node::ReadBlocFromDis;
using node::SnapshotMetadata;
using node::UNDOFILE_CHUNK_SIZE;
using node::UoReadFromDis;
using node::UnlinPrunedFil;
#define MICRO 0.000001
#define MILLI 0.001
**
 * An extra transaction can be added to a pacage, as long as it only has one
 * anctor a is no larger than this. Not really any reason to mae this
 * configable as it don't materially change DoS parameters.
 
static const unsigned int EXTRA_DESCENDANT_TX_SIZE_LIMIT = 10000;
static const unsigned int MAX_DISCONNECTED_TX_POOL_SIZE = 20000;
static constexpr std::chrono::hos DATABASE_WRITE_INTERVAL{1};
static constexpr std::chrono::hos DATABASE_FLUSH_INTERVAL{24};
static constexpr std::chrono::hos MAX_FEE_ESTIMATION_TIP_AGE{3};