Profile shouldn't own DesktopNotificationService.

chrome/browser/notifications/desktop_notification_service_factory.h

+// Copyright (c) 2011 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_NOTIFICATIONS_DESKTOP_NOTIFICATION_SERVICE_FACTORY_H_
+#define CHROME_BROWSER_NOTIFICATIONS_DESKTOP_NOTIFICATION_SERVICE_FACTORY_H_
+
+#include "base/memory/singleton.h"
+#include "chrome/browser/profiles/profile_keyed_service_factory.h"
+
+class DesktopNotificationService;
+class Profile;
+
+// Singleton that owns all DesktopNotificationServices and associates them with
+// Profiles. Listens for the Profile's destruction notification and cleans up
+// the associated DesktopNotificationService.
+class DesktopNotificationServiceFactory : public ProfileKeyedServiceFactory {
+ public:
+  // Returns the DesktopNotificationService that provides desktop notifications
+  // for |profile|.
+  static DesktopNotificationService* GetForProfile(Profile* profile);
+
+  // Forces an association between |profile| and |service|. Used in unit tests
+  // where we need a mock DesktopNotificationService.
+  static void ForceAssociationBetween(Profile* profile,
+                                      DesktopNotificationService* service);
+
+  static DesktopNotificationServiceFactory* GetInstance();
+
+ private:
+  friend struct DefaultSingletonTraits<DesktopNotificationServiceFactory>;
+
+  DesktopNotificationServiceFactory();
+  virtual ~DesktopNotificationServiceFactory();
+
+  // ProfileKeyedServiceFactory:
+  virtual ProfileKeyedService* BuildServiceInstanceFor(Profile* profile) const;
+  // Use a separate desktop notification service for incognito.
+  virtual bool ServiceHasOwnInstanceInIncognito();
+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_DESKTOP_NOTIFICATION_SERVICE_FACTORY_H_