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);
+
+  static DesktopNotificationServiceFactory* GetInstance();
+
+ private:
+  friend struct DefaultSingletonTraits<DesktopNotificationServiceFactory>;
+
+  DesktopNotificationServiceFactory();
+  virtual ~DesktopNotificationServiceFactory();
+
+  // ProfileKeyedServiceFactory:
+  virtual ProfileKeyedService* BuildServiceInstanceFor(Profile* profile) const;

[Avi (use Gerrit), 2011/04/12 13:57:15]

OVERRIDE

[Torne, 2011/04/12 15:22:12]

Done.

+  // Use a separate desktop notification service for incognito.
+  virtual bool ServiceHasOwnInstanceInIncognito();

[Avi (use Gerrit), 2011/04/12 13:57:15]

OVERRIDE

[Torne, 2011/04/12 15:22:12]

Done.

+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_DESKTOP_NOTIFICATION_SERVICE_FACTORY_H_