Change ProfileDownloader to use AccountTrackerService.

components/signin/core/browser/account_tracker_service.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_TRACKER_SERVICE_H_
 #define COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_TRACKER_SERVICE_H_
 
 #include <list>
 #include <map>
 #include <string>
@@ skipping 23 matching lines @@
  public:
   // Name of the preference property that persists the account information
   // tracked by this service.
   static const char kAccountInfoPref[];
 
   // TODO(mlerman): Remove all references to Profile::kNoHostedDomainFound in
   // favour of this.
   // Value representing no hosted domain in the kProfileHostedDomain preference.
   static const char kNoHostedDomainFound[];
 
+  // Value representing no picture URL associated with and account.

[Mike Lerman, 2015/04/30 19:09:07]

nit: s/and/an/

[anthonyvd, 2015/05/01 19:16:34]

Done.

+  static const char kNoPictureURLFound[];
+
   // Information about a specific account.
   struct AccountInfo {
     AccountInfo();
     ~AccountInfo();
 
     std::string account_id;  // The account ID used by OAuth2TokenService.
     std::string gaia;
     std::string email;
     std::string full_name;
     std::string given_name;
     std::string hosted_domain;
     std::string locale;
-    // TODO(rogerta): eventually this structure will include other information
-    // about the account, like full name, profile picture URL, etc.
+    std::string picture_url;
+    // TODO(anthonyvd): Eventually this structure should contain the account
+    // service flags.
 
-    bool IsValid();
+    bool IsValid() const;
   };
 
   // Clients of AccountTrackerService can implement this interface and register
   // with AddObserver() to learn about account information changes.
   class Observer {
    public:
     virtual ~Observer() {}
     virtual void OnAccountUpdated(const AccountInfo& info) {}
     virtual void OnAccountUpdateFailed(const std::string& account_id) {}
     virtual void OnAccountRemoved(const AccountInfo& info) {}
@@ skipping 106 matching lines @@
   std::list<std::string> pending_user_info_fetches_;
 
   // Holds references to refresh token annotation requests keyed by account_id.
   base::ScopedPtrHashMap<std::string, RefreshTokenAnnotationRequest>
       refresh_token_annotation_requests_;
 
   DISALLOW_COPY_AND_ASSIGN(AccountTrackerService);
 };
 
 #endif  // COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_TRACKER_SERVICE_H_