Have AutofillManagerDelegate supply the AutofillWebDataService to core code.

components/autofill/core/browser/personal_data_manager.h

 // Copyright 2013 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_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_
 
 #include <set>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/observer_list.h"
+#include "base/prefs/pref_service.h"
 #include "base/strings/string16.h"
+#include "components/autofill/core/browser/autofill_metrics.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/autofill/core/browser/field_types.h"
+#include "components/autofill/core/browser/webdata/autofill_webdata_service.h"

[Ilya Sherman, 2013/11/14 23:45:55]

These should remain as forward-declarations in the header file.

[blundell, 2013/11/15 16:04:54]

Done for PrefService. The other two changes are needed due to the
scoped_(ref)ptr operations now happening in the .h file.

On 2013/11/14 23:45:55, Ilya Sherman wrote:

 #include "components/autofill/core/browser/webdata/autofill_webdata_service_observer.h"
 #include "components/webdata/common/web_data_service_consumer.h"
 
-class PrefService;
 class RemoveAutofillTester;
 
-namespace content {
-class BrowserContext;
-}
-
 namespace autofill {
 class AutofillInteractiveTest;
-class AutofillMetrics;
 class AutofillTest;
 class FormStructure;
 class PersonalDataManagerObserver;
 class PersonalDataManagerFactory;
 }  // namespace autofill
 
 namespace autofill_helper {
 void SetProfiles(int, std::vector<autofill::AutofillProfile>*);
 void SetCreditCards(int, std::vector<autofill::CreditCard>*);
 }  // namespace autofill_helper
 
 namespace autofill {
 
 // Handles loading and saving Autofill profile information to the web database.
 // This class also stores the profiles loaded from the database for use during
 // Autofill.
 class PersonalDataManager : public WebDataServiceConsumer,
                             public AutofillWebDataServiceObserverOnUIThread {
  public:
   // A pair of GUID and variant index. Represents a single FormGroup and a
   // specific data variant.
   typedef std::pair<std::string, size_t> GUIDPair;
 
   explicit PersonalDataManager(const std::string& app_locale);
   virtual ~PersonalDataManager();
 
-  // Kicks off asynchronous loading of profiles and credit cards. |context| and
-  // |pref_service| must outlive this instance. |is_off_the_record| informs
+  // Kicks off asynchronous loading of profiles and credit cards.
+  // |pref_service| must outlive this instance.  |is_off_the_record| informs
   // this instance whether the user is currently operating in an off-the-record
   // context.
-  void Init(content::BrowserContext* context,
+  void Init(scoped_refptr<AutofillWebDataService> database,
             PrefService* pref_service,
             bool is_off_the_record);
 
   // WebDataServiceConsumer:
   virtual void OnWebDataServiceRequestDone(
       WebDataServiceBase::Handle h,
       const WDTypedResult* result) OVERRIDE;
 
   // AutofillWebDataServiceObserverOnUIThread:
   virtual void AutofillMultipleChanged() OVERRIDE;
@@ skipping 179 matching lines @@
   void CancelPendingQuery(WebDataServiceBase::Handle* handle);
 
   // The first time this is called, logs an UMA metrics for the number of
   // profiles the user has. On subsequent calls, does nothing.
   void LogProfileCount() const;
 
   // Returns the value of the AutofillEnabled pref.
   virtual bool IsAutofillEnabled() const;
 
   // For tests.
-  const AutofillMetrics* metric_logger() const;
-  void set_metric_logger(const AutofillMetrics* metric_logger);
-  void set_browser_context(content::BrowserContext* context);
-  void set_pref_service(PrefService* pref_service);
+  const AutofillMetrics* metric_logger() const { return metric_logger_.get(); }
 
-  // The browser context this PersonalDataManager is in.
-  content::BrowserContext* browser_context_;
+  void set_database(scoped_refptr<AutofillWebDataService> database) {
+    database_ = database;
+  }
+
+  void set_metric_logger(const AutofillMetrics* metric_logger) {
+    metric_logger_.reset(metric_logger);
+  }
+
+  void set_pref_service(PrefService* pref_service) {
+    pref_service_ = pref_service;
+  }
+
+  // The backing database that this PersonalDataManager uses.
+  scoped_refptr<AutofillWebDataService> database_;
 
   // True if personal data has been loaded from the web database.
   bool is_data_loaded_;
 
   // The loaded web profiles.
   ScopedVector<AutofillProfile> web_profiles_;
 
   // Auxiliary profiles.
   mutable ScopedVector<AutofillProfile> auxiliary_profiles_;
 
@@ skipping 36 matching lines @@
 
   // Whether we have already logged the number of profiles this session.
   mutable bool has_logged_profile_count_;
 
   DISALLOW_COPY_AND_ASSIGN(PersonalDataManager);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_