Use new people.get api instead of oauth2/v1/userinfo.

components/policy/core/common/cloud/user_info_fetcher.h

Index: components/policy/core/common/cloud/user_info_fetcher.h
diff --git a/components/policy/core/common/cloud/user_info_fetcher.h b/components/policy/core/common/cloud/user_info_fetcher.h
index c248d81b06852ccdddc8f5318a6fd8ed5f724d0b..0f4c25f533daa17808e3a237945ca1e26ee2b42e 100644
--- a/components/policy/core/common/cloud/user_info_fetcher.h
+++ b/components/policy/core/common/cloud/user_info_fetcher.h
@@ -8,6 +8,7 @@
 #include <string>
 #include "base/memory/scoped_ptr.h"
 #include "components/policy/policy_export.h"
+#include "google_apis/gaia/gaia_oauth_client.h"
 #include "net/url_request/url_fetcher_delegate.h"
 
 class GoogleServiceAuthError;
@@ -25,7 +26,7 @@ namespace policy {
 
 // Class that makes a UserInfo request, parses the response, and notifies
 // a provided Delegate when the request is complete.
-class POLICY_EXPORT UserInfoFetcher : public net::URLFetcherDelegate {
+class POLICY_EXPORT UserInfoFetcher {
  public:
   class POLICY_EXPORT Delegate {
    public:
@@ -48,13 +49,25 @@ class POLICY_EXPORT UserInfoFetcher : public net::URLFetcherDelegate {
   // Starts the UserInfo request, using the passed OAuth2 |access_token|.
   void Start(const std::string& access_token);
 
-  // net::URLFetcherDelegate implementation.
-  virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
-
  private:
-  Delegate* delegate_;
-  net::URLRequestContextGetter* context_;
-  scoped_ptr<net::URLFetcher> url_fetcher_;
+  class GaiaDelegate : public gaia::GaiaOAuthClient::Delegate {

[Andrew T Wilson (Slow), 2014/04/27 22:58:15]

This is fine as-is. I have a slight preference for the pattern where this .h
file just has a forward declaration/scoped_ptr:

class GaiaDelegate;
scoped_ptr<GaiaDelegate> delegate_;

...and we define + instantiate the class in the .cc file, hiding the internals.
But it's OK if you keep it like this too.

+   public:
+    explicit GaiaDelegate(UserInfoFetcher::Delegate* delegate);
+
+   private:
+    // gaia::GaiaOAuthClient::Delegate implementation.
+    virtual void OnGetUserInfoResponse(
+        scoped_ptr<base::DictionaryValue> user_info) OVERRIDE;
+    virtual void OnOAuthError() OVERRIDE;
+    virtual void OnNetworkError(int response_code) OVERRIDE;
+
+    UserInfoFetcher::Delegate* delegate_;
+
+    DISALLOW_COPY_AND_ASSIGN(GaiaDelegate);
+  };
+
+  GaiaDelegate delegate_;
+  gaia::GaiaOAuthClient gaia_client_;
 
   DISALLOW_COPY_AND_ASSIGN(UserInfoFetcher);
 };