Move reset state to a more appropriate location

Summary: Not really part of the connect process in this case, so is getting moved closer to where the CSR is generated.

Reviewed By: passy

Differential Revision: D46849966

fbshipit-source-id: b91e2925552cd7fcab67b3d1d7af006f4f8c1431

xplat/Flipper/FlipperConnectionManagerImpl.cpp

@@ -225,11 +225,8 @@ bool FlipperConnectionManagerImpl::connectAndExchangeCertificate() {
 
   connectingInsecurely->complete();
 
-  auto resettingState = flipperState_->start("Reset state");
-  contextStore_->resetState();
-  resettingState->complete();
-
   requestSignedCertificate();
+
   return true;
 }
 
@@ -464,6 +461,10 @@ void FlipperConnectionManagerImpl::processSignedCertificateResponse(
 }
 
 void FlipperConnectionManagerImpl::requestSignedCertificate() {
+  auto resettingState = flipperState_->start("Reset connection store state");
+  contextStore_->resetState();
+  resettingState->complete();
+
   auto generatingCSR = flipperState_->start("Generate CSR");
   std::string csr = contextStore_->getCertificateSigningRequest();
   generatingCSR->complete();