[Merge M56] arc: Fix situation when ARC cannot be disabled safely.

chrome/browser/chromeos/arc/arc_auth_service.cc

Index: chrome/browser/chromeos/arc/arc_auth_service.cc
diff --git a/chrome/browser/chromeos/arc/arc_auth_service.cc b/chrome/browser/chromeos/arc/arc_auth_service.cc
index 9dc63e38014cfbd176f14b1e937a35a1e99c9de5..fae06a64a0ff0aa8750d41ee1a977b73c8faf138 100644
--- a/chrome/browser/chromeos/arc/arc_auth_service.cc
+++ b/chrome/browser/chromeos/arc/arc_auth_service.cc
@@ -505,6 +505,17 @@ void ArcAuthService::OnSignInFailed(mojom::ArcSignInFailureReason reason) {
 void ArcAuthService::OnProvisioningFinished(ProvisioningResult result) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
+  // If the Mojo message to notify finishing the provisioning is already sent
+  // from the container, it will be processed even after requesting to stop the
+  // container. Ignore all |result|s arriving while ARC is disabled, in order to
+  // avoid popping up an error message triggered below. This code intentionally
+  // does not support the case of reenabling.
+  if (!IsArcEnabled()) {
+    LOG(WARNING) << "Provisioning result received after Arc was disabled. "
+                 << "Ignoring result " << static_cast<int>(result) << ".";
+    return;
+  }
+
   // Due asynchronous nature of stopping Arc bridge, OnProvisioningFinished may
   // arrive after setting the |State::STOPPED| state and |State::Active| is not
   // guaranty set here. prefs::kArcDataRemoveRequested is also can be active