Merge provisioning success/error handling.

chrome/browser/chromeos/arc/arc_auth_service.h

Index: chrome/browser/chromeos/arc/arc_auth_service.h
diff --git a/chrome/browser/chromeos/arc/arc_auth_service.h b/chrome/browser/chromeos/arc/arc_auth_service.h
index 9ba105cbeb00e2be42807b3d5b3606c3e8485f2b..f40a80934e5fff6012301f7695fb3ce476478833 100644
--- a/chrome/browser/chromeos/arc/arc_auth_service.h
+++ b/chrome/browser/chromeos/arc/arc_auth_service.h
@@ -181,8 +181,6 @@ class ArcAuthService : public ArcService,
   void GetIsAccountManagedDeprecated(
       const GetIsAccountManagedDeprecatedCallback& callback) override;
 
-  void OnSignInFailedInternal(ProvisioningResult result);
-
   // Called from Arc support platform app to set auth code and start arc.
   void OnAuthCodeObtained(const std::string& auth_code);
 
@@ -244,6 +242,8 @@ class ArcAuthService : public ArcService,
 
   void StartArc();
 
+  void OnProvisioningFinished(ProvisioningResult result);

[Luis Héctor Chávez, 2016/11/15 23:40:50]

Can this be private?

[hidehiko, 2016/11/17 11:04:54]

In this CL, we can move it to private. However, in a following CL you've already
LGTM'ed, we need to move back to public. So let me keep this in public.

+
  private:
   using AccountInfoCallback = base::Callback<void(mojom::AccountInfoPtr)>;
   class AccountInfoNotifier;