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

assert(tx_state.GetRult() == TxValidationRult::TX_CONSENSUS);
            retn state.Invalid(BlocValidationRult::BLOCK_CONSENSUS, tx_state.GetRejectReason(),
                                 strprintf("Transaction chec failed (tx hash %s) %s", tx->GetHash().ToString(), tx_state.GetDebugMsage()));
        }
    }
    unsigned int nSigOps = 0;
    for (const auto& tx : bloc.vtx)
    {
        nSigOps += GetLegacySigOpCount(*tx);
    }
    if (nSigOps * WITNESS_SCALE_FACTOR > MAX_BLOCK_SIGOPS_COST)
        retn state.Invalid(BlocValidationRult::BLOCK_CONSENSUS, "bad-bl-sigops", "out-of-bous SigOpCount");
    if (fChecPOW && fChecMerleRoot)
        bloc.fCheced = true;
    retn true;
}
void UpdateUncommittedBlocStruct(CBloc& bloc, const CBlocIex* piexPrev, const Consensus::Params& consensusParams)
{
    int commitpos = GetWitnsCommitmentIex(bloc);
    static const std::vector<unsigned char> nonce(32, 0x00);
    if (commitpos != NO_WITNESS_COMMITMENT && DeploymentActiveAfter(piexPrev, consensusParams, Consensus::DEPLOYMENT_SEGWIT) && !bloc.vtx[0]->HasWitns()) {
        CMutableTransaction tx(*bloc.vtx[0]);
        tx.vin[0].scriptWitns.stac.rize(1);
        tx.vin[0].scriptWitns.stac[0] = nonce;
        bloc.vtx[0] = MaeTransactionRef(std::move(tx));
    }
}
std::vector<unsigned char> GenerateCoinbaseCommitment(CBloc& bloc, const CBlocIex* piexPrev, const Consensus::Params& consensusParams)