Refactor notifications of chrome/browser/webdata

chrome/browser/webdata/autofill_web_data_service_impl.h

 // Copyright (c) 2012 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 CHROME_BROWSER_WEBDATA_AUTOFILL_WEB_DATA_SERVICE_IMPL_H_
 #define CHROME_BROWSER_WEBDATA_AUTOFILL_WEB_DATA_SERVICE_IMPL_H_
 
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/api/webdata/autofill_web_data_service.h"
 
@@ skipping 23 matching lines @@
   virtual WebDataServiceBase::Handle GetAutofillProfiles(
       WebDataServiceConsumer* consumer) OVERRIDE;
   virtual void AddCreditCard(const CreditCard& credit_card) OVERRIDE;
   virtual void UpdateCreditCard(const CreditCard& credit_card) OVERRIDE;
   virtual void RemoveCreditCard(const std::string& guid) OVERRIDE;
   virtual WebDataServiceBase::Handle
       GetCreditCards(WebDataServiceConsumer* consumer) OVERRIDE;
 
   // WebDataServiceBase implementation.
   virtual void CancelRequest(Handle h) OVERRIDE;
-  virtual content::NotificationSource GetNotificationSource() OVERRIDE;
+
+  // AutofillWebDataService implementation.

[dhollowa, 2013/03/12 22:22:28]

Please confirm with joi@ that this conforms to his plans for componentization of
this stuff.

[Jói, 2013/03/12 22:28:18]

Yeah, this is OK, although once we get further along the class hierarchy will
collapse a bit and these won't be virtual. Once we finish componentizing, there
will just be one AutofillWebDataService class I think, which will inherit from a
base class for all XyzWebDataService classes, probably named WebDataServiceBase
or just WebDataService.

+  virtual void AddObserver(WebDataServiceObserver* observer) OVERRIDE;
+  virtual void RemoveObserver(WebDataServiceObserver* observer) OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(AutofillWebDataServiceImpl);
 
   const scoped_refptr<WebDataService> service_;
 };
 
 #endif  // CHROME_BROWSER_WEBDATA_AUTOFILL_WEB_DATA_SERVICE_IMPL_H_