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

chrome/browser/extensions/install_tracker_factory.cc

Index: chrome/browser/extensions/install_tracker_factory.cc
diff --git a/chrome/browser/extensions/install_tracker_factory.cc b/chrome/browser/extensions/install_tracker_factory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..af84071638493cf21dc54a81edc36ca6264e0572
--- /dev/null
+++ b/chrome/browser/extensions/install_tracker_factory.cc
@@ -0,0 +1,46 @@
+// Copyright (c) 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_factory.h"
+
+#include "chrome/browser/extensions/install_tracker.h"
+#include "chrome/browser/profiles/profile_dependency_manager.h"
+
+namespace extensions {
+
+// static
+InstallTracker* InstallTrackerFactory::GetForProfile(Profile* profile) {
+  return static_cast<InstallTracker*>(
+      GetInstance()->GetServiceForProfile(profile, true));
+}
+
+// static
+void InstallTrackerFactory::ResetForProfile(Profile* profile) {
+  InstallTrackerFactory* factory = GetInstance();
+  factory->ProfileShutdown(profile);
+  factory->ProfileDestroyed(profile);
+}
+
+InstallTrackerFactory* InstallTrackerFactory::GetInstance() {
+  return Singleton<InstallTrackerFactory>::get();
+}
+
+InstallTrackerFactory::InstallTrackerFactory()
+    : ProfileKeyedServiceFactory("InstallTracker",
+                                 ProfileDependencyManager::GetInstance()) {
+}
+
+InstallTrackerFactory::~InstallTrackerFactory() {
+}
+
+ProfileKeyedService* InstallTrackerFactory::BuildServiceInstanceFor(
+    Profile* profile) const {
+  return new InstallTracker(profile);
+}
+
+bool InstallTrackerFactory::ServiceIsCreatedWithProfile() const {
+  return true;

[xiyuan, 2013/02/21 06:33:17]

Why InstallTracker needs to be created with profile? Shouldn't the default lazy
init work?

[koz (OOO until 15th September), 2013/02/25 02:39:28]

Yes, true. Removed.

+}
+
+}  // namespace extensions