Upstream iOS implementation of ProfileOAuth2TokenService

components/signin/core/browser/signin_client.h

Index: components/signin/core/browser/signin_client.h
diff --git a/components/signin/core/browser/signin_client.h b/components/signin/core/browser/signin_client.h
index 4b8837ce65dd4c451265487ab11dd974259014f6..4187f475cb76139f460ca50c02f86f5de8e8c186 100644
--- a/components/signin/core/browser/signin_client.h
+++ b/components/signin/core/browser/signin_client.h
@@ -17,6 +17,14 @@ class CanonicalCookie;
 class URLRequestContextGetter;
 }
 
+#if defined(OS_IOS)
+namespace ios {
+// TODO(msarda): http://crbug.com/358544 Remove this iOS specific code from the
+// core SigninClient.

[Roger Tawa OOO till Jul 10th, 2014/04/10 15:04:08]

Is this a bug you plan to fix soon?  Can you put more info in the bug as to why
this is here now and why you want to eventually remove it :-)

[msarda, 2014/04/10 15:13:36]

No, this is low priority. It was something Colin wanted me to add in here, but
this will become clearer if I need to remove once we do more componentization
and understand what is the best strategy to use for mutli-platform components.

[blundell, 2014/04/10 15:17:58]

Ideally the client wouldn't have anything platform-specific. However, the
situation around ProfileOAuth2TokenService is challenging due the tricky
inheritance issues, and I don't want to block the high-value work of upstreaming
PO2TSIOS on  having to figure out those issues. As Mihai says, it should become
more clear over time what the right situation is here (and maybe Alan's work
will help, actually).

On 2014/04/10 15:13:36, msarda wrote:

+class ProfileOAuth2TokenServiceIOSProvider;
+}
+#endif
+
 // An interface that needs to be supplied to the Signin component by its
 // embedder.
 class SigninClient {
@@ -57,6 +65,12 @@ class SigninClient {
   // Called when Google signin has succeeded.
   virtual void GoogleSigninSucceeded(const std::string& username,
                                      const std::string& password) {}
+
+#if defined(OS_IOS)
+  // TODO(msarda): http://crbug.com/358544 Remove this iOS specific code from
+  // the core SigninClient.
+  virtual ios::ProfileOAuth2TokenServiceIOSProvider* GetIOSProvider() = 0;
+#endif
 };
 
 #endif  // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_CLIENT_H_