Synced Notification Sync Change Processor

chrome/browser/notifier/chrome_notifier_service_factory.cc

+// 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.
+
+#include "chrome/browser/notifier/chrome_notifier_service_factory.h"
+
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/notifier/chrome_notifier_service.h"
+#include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/profiles/profile_dependency_manager.h"
+
+// static
+ChromeNotifierService* ChromeNotifierServiceFactory::GetForProfile(
+    Profile* profile, Profile::ServiceAccessType sat) {
+  return static_cast<ChromeNotifierService*>(
+      GetInstance()->GetServiceForProfile(profile, true));
+}
+
+// static
+ChromeNotifierService*
+ChromeNotifierServiceFactory::GetForProfileIfExists(
+    Profile* profile, Profile::ServiceAccessType sat) {
+  return static_cast<ChromeNotifierService*>(
+      GetInstance()->GetServiceForProfile(profile, false));
+}
+
+// static
+ChromeNotifierService*
+ChromeNotifierServiceFactory::GetForProfileWithoutCreating(Profile* profile) {
+  return static_cast<ChromeNotifierService*>(
+      GetInstance()->GetServiceForProfile(profile, false));
+}
+
+// static
+ChromeNotifierServiceFactory* ChromeNotifierServiceFactory::GetInstance() {
+  return Singleton<ChromeNotifierServiceFactory>::get();
+}
+
+ChromeNotifierServiceFactory::ChromeNotifierServiceFactory()
+    : ProfileKeyedServiceFactory(
+          "ChromeNotifierService", ProfileDependencyManager::GetInstance()) {}
+
+ChromeNotifierServiceFactory::~ChromeNotifierServiceFactory() {
+}
+
+ProfileKeyedService*
+ChromeNotifierServiceFactory::BuildServiceInstanceFor(Profile* profile) const {
+  // TODO: This assumes the notification_ui_manager remains valid throughout the

[dcheng, 2013/01/18 21:56:13]

TODO's should have an associated name.

[Pete Williamson, 2013/01/23 01:45:55]

Instead, I just did the research and removed the TODO (I use a personal
convention of not putting my name on anything that should be done before
checkin, and this one slipped past me.)

+  //  session.  Is that a good assumption?
+  NotificationUIManager* notification_manager =
+      g_browser_process->notification_ui_manager();
+  ChromeNotifierService* chrome_notifier_service =
+      new ChromeNotifierService(profile, notification_manager);
+  return chrome_notifier_service;
+}
+
+bool ChromeNotifierServiceFactory::ServiceRedirectedInIncognito() const {
+  return true;

[dcheng, 2013/01/18 21:56:13]

It might be helpful to comment why we can't just use the default here.

[Pete Williamson, 2013/01/23 01:45:55]

Started an email thread to find the right setting and the reason for it.

+}
+
+bool ChromeNotifierServiceFactory::ServiceIsNULLWhileTesting() const {
+  return true;
+}
+
+bool ChromeNotifierServiceFactory::ServiceIsCreatedWithProfile() const {
+  return true;
+}