[Merge-M56] arc: Fix double provisioning failure reporting.

chrome/browser/chromeos/arc/arc_auth_service_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <memory>
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "chrome/browser/chromeos/arc/arc_auth_service.h"
+#include "chrome/browser/chromeos/arc/arc_optin_uma.h"
 #include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h"
 #include "chrome/browser/chromeos/login/users/scoped_user_manager_enabler.h"
 #include "chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/prefs/pref_service_syncable_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
 #include "chrome/browser/ui/ash/multi_user/multi_user_util.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_profile.h"
@@ skipping 403 matching lines @@
       WaitForDataRemoved(ArcAuthService::State::SHOWING_TERMS_OF_SERVICE));
 
   EXPECT_FALSE(
       profile()->GetPrefs()->GetBoolean(prefs::kArcDataRemoveRequested));
 
   auth_service()->StartArc();
   EXPECT_EQ(ArcAuthService::State::ACTIVE, auth_service()->state());
   auth_service()->Shutdown();
 }
 
+TEST_F(ArcAuthServiceTest, IgnoreSecondErrorReporting) {
+  profile()->GetPrefs()->SetBoolean(prefs::kArcEnabled, true);
+  auth_service()->OnPrimaryUserProfilePrepared(profile());
+  auth_service()->StartArc();
+  EXPECT_EQ(ArcAuthService::State::ACTIVE, auth_service()->state());
+
+  // Report some failure that does not stop the bridge.
+  auth_service()->OnProvisioningFinished(
+      ProvisioningResult::GMS_SIGN_IN_FAILED);
+  EXPECT_EQ(ArcAuthService::State::ACTIVE, auth_service()->state());
+
+  // Try to send another error that stops the bridge if sent first. It should
+  // be ignored.
+  auth_service()->OnProvisioningFinished(
+      ProvisioningResult::CHROME_SERVER_COMMUNICATION_ERROR);
+  EXPECT_EQ(ArcAuthService::State::ACTIVE, auth_service()->state());
+
+  auth_service()->Shutdown();
+}
+
 }  // namespace arc