Fix premature installation completion signal in Bloom installer. This CL removes calls to `status_reporter_.Complete(**true**, success_msg)` within the `VerifyBundleSignature` and `GetOwnerConfig` methods. These functions represent intermediate steps in the installation process, not its completion, so we shouldn't mark the installation process as complete. PiperOrigin-RevId: 930656122 Change-Id: Iafbbbb270d44ab73e1fe44b3312dce5d6eb81892 (cherry picked from commit ba359d6dda983c72ec15d52602e8316d8950a74d)
diff --git a/install/installer.cc b/install/installer.cc index 6b1e7d3..bc03968 100644 --- a/install/installer.cc +++ b/install/installer.cc
@@ -266,7 +266,6 @@ const std::string success_msg = "Bundle signature check successful."; logger_.Log(State::kSuccess, absl::Now() - check_start_time, success_msg, LogCode::kBundleSignatureSuccess); - status_reporter_.Complete(true, success_msg); return true; } @@ -330,7 +329,6 @@ const std::string success_msg = "Successfully retrieved owner configuration"; logger_.Log(State::kSuccess, absl::Now() - start_time, success_msg, LogCode::kGetOwnerConfigSuccess); - status_reporter_.Complete(true, success_msg); return config; }