Nuke NotificationUIManager from PlatformNotificationServiceImpl

chrome/browser/notifications/notification_display_service_factory.h

+// Copyright 2016 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_NOTIFICATION_DISPLAY_SERVICE_FACTORY_H_
+#define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_FACTORY_H_
+
+#include "base/macros.h"
+#include "chrome/browser/notifications/notification_display_service.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "components/keyed_service/content/browser_context_keyed_service_factory.h"

[Peter Beverloo, 2016/04/18 14:57:10]

nit: Forward declare the NotificationDisplayService.
nit: I don't think we have to include browser_context_dependency_manager.h here.

[Miguel Garcia, 2016/04/19 14:24:57]

Done.

+
+class NotificationDisplayServiceFactory
+    : public BrowserContextKeyedServiceFactory {
+ public:
+  NotificationDisplayServiceFactory();
+
+  static NotificationDisplayService* GetForProfile(Profile* profile);
+  static NotificationDisplayServiceFactory* GetInstance();
+
+ private:
+  // BrowserContextKeyedBaseFactory implementation.
+  KeyedService* BuildServiceInstanceFor(
+      content::BrowserContext* context) const override;
+  content::BrowserContext* GetBrowserContextToUse(
+      content::BrowserContext* context) const override;
+};

[Peter Beverloo, 2016/04/18 14:57:10]

DISALLOW_COPY_AND_ASSIGN(NotificationDisplayServiceFactory);

[Miguel Garcia, 2016/04/19 14:24:57]

Done.

+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_FACTORY_H_