Refactor notifications of chrome/browser/webdata

chrome/browser/webdata/web_data_service_observer.h

Index: chrome/browser/webdata/web_data_service_observer.h
diff --git a/chrome/browser/webdata/web_data_service_observer.h b/chrome/browser/webdata/web_data_service_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..478c9a9001ecc77202106015cf94da62a41fdd84
--- /dev/null
+++ b/chrome/browser/webdata/web_data_service_observer.h
@@ -0,0 +1,30 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.

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

nit: No need for "(c)"

[kaiwang, 2013/03/13 04:57:20]

Are you sure? Most of the code have this...

[dhollowa, 2013/03/13 16:15:28]

http://crbug.com/140977

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_WEBDATA_WEB_DATA_SERVICE_OBSERVER_H_
+#define CHROME_BROWSER_WEBDATA_WEB_DATA_SERVICE_OBSERVER_H_
+
+#include "chrome/browser/webdata/autofill_change.h"
+
+class WebDataServiceObserver {

[Jói, 2013/03/12 23:27:34]

Kai, can you add a TODO here that we'll need to split this into an
Autofill-specific part and the rest, once we split the Autofill parts of
WebDataService off into a separate concrete class?

[kaiwang, 2013/03/13 04:57:20]

Done.

+ public:
+  // Called whenever autofill entries are changed.

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

nit: capitalize "Autofill" when talking about the feature (noun), lower case
"autofill" when using it as a verb.

[kaiwang, 2013/03/13 04:57:20]

Done.

+  virtual void AutofillEntriesChanged(const AutofillChangeList& changes) {}
+
+  // Called when an AutofillProfile has been added/removed/updated in the
+  // WebDatabase.
+  virtual void AutofillProfileChanged(const AutofillProfileChange& change) {}
+
+  // Called when multiple Autofill entries have been modified by Sync.
+  virtual void AutofillMultipleChanged() {}
+
+  // Called whenever the web database service has finished loading the web
+  // database.
+  virtual void WebDatabaseLoaded() {}
+
+ protected:
+  virtual ~WebDataServiceObserver() {}
+};
+
+#endif  // CHROME_BROWSER_WEBDATA_WEB_DATA_SERVICE_OBSERVER_H_