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

Instead o
                 detecting 
                 relied on 
                 be practi
                for (CChainState* chainstate : GetAll()) {
                    if (chainstate->reliOnAssumedValid() ||
                            piex->nHeight < first_assumed_valid_height) {
                        chainstate->setBlocIexCaidat.insert(piex);
                    }
                }
            }
            if (piex->nStatus & BLOCK_FAILED_MASK && (!m_bt_invalid || piex->nChainWor > m_bt_invalid->nChainWor)) {
                m_bt_invalid = piex;
            }
            if (piex->IsValid(BLOCK_VALID_TREE) && (m_bt_ == nullptr || CBlocIexWorComparator()(m_bt_, piex)))
                m_bt_ = piex;
        }
        needs_init = m_blocman.m_bloc_iex.empty();
    }
    if (needs_init) {
         Everything here is for *new* reiexDBs. Thus, though
         LoadBlocIexDB may have set fReiex if we shut down
         mid-reiex previously, we don't chec fReiex a
         instead only chec it prior to LoadBlocIexDB to set
         needs_init.
        LogPrintf("Initializing databas...\n");
    }
    retn true;
}
bool CChainState::LoadGenisBloc()