Change NotifyAppList*() functions into observers on a ProfileKeyedService.

chrome/browser/extensions/install_tracker.cc

Index: chrome/browser/extensions/install_tracker.cc
diff --git a/chrome/browser/extensions/install_tracker.cc b/chrome/browser/extensions/install_tracker.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0734346eb06ff8eb01ac82c06dd7de52d3ccc861
--- /dev/null
+++ b/chrome/browser/extensions/install_tracker.cc
@@ -0,0 +1,47 @@
+// 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.
+
+#include "chrome/browser/extensions/install_tracker.h"
+
+namespace extensions {
+
+InstallTracker::InstallTracker() {
+}
+
+InstallTracker::~InstallTracker() {
+}
+
+void InstallTracker::OnBeginExtensionInstall(
+    const std::string& extension_id,
+    const std::string& extension_name,
+    const gfx::ImageSkia& installing_icon,
+    bool is_app) {
+  FOR_EACH_OBSERVER(InstallObserver, observers_,
+      OnBeginExtensionInstall(extension_id,

[tfarina, 2013/02/25 23:43:02]

If it fits I'd align with InstallObserver. Up to you though.

[koz (OOO until 15th September), 2013/02/26 00:10:51]

Done.

+                              extension_name,
+                              installing_icon,
+                              is_app));
+}
+
+void InstallTracker::OnDownloadProgress(const std::string& extension_id,
+                                        int percent_downloaded) {
+  FOR_EACH_OBSERVER(InstallObserver, observers_,
+      OnDownloadProgress(extension_id, percent_downloaded));
+}
+
+void InstallTracker::OnInstallFailure(
+    const std::string& extension_id) {
+  FOR_EACH_OBSERVER(InstallObserver, observers_,
+      OnInstallFailure(extension_id));

[tfarina, 2013/02/25 23:43:02]

If it fits I'd align with InstallObserver. Up to you though.

[koz (OOO until 15th September), 2013/02/26 00:10:51]

Done.

+}
+
+void InstallTracker::AddObserver(InstallObserver* observer) {

[tfarina, 2013/02/25 23:43:02]

the order doesn't match with the order declared in the header file, this and
RemoveObserver should be moved above OnBeginExtensionInstall

[koz (OOO until 15th September), 2013/02/26 00:10:51]

Done.

+  observers_.AddObserver(observer);
+}
+
+void InstallTracker::RemoveObserver(InstallObserver* observer) {
+  observers_.RemoveObserver(observer);
+}
+
+}  // namespace extensions