Nuke NotificationUIManager from PlatformNotificationServiceImpl

chrome/browser/notifications/notification_bridge.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_BRIDGE_H_
+#define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_BRIDGE_H_
+
+#include <set>
+#include <string>
+
+#include "base/macros.h"
+
+class Notification;
+
+// Provides the low-level interface that enables notifications to be displayed
+// and interacted with on the user's screen, orthogonal of whether this
+// functionality is provided by the browser or by the operating system.
+// TODO(miguelg): Add support for click and close events.
+class NotificationBridge {

[Peter Beverloo, 2016/04/20 17:34:08]

Sorry for the bike-shedding, but what do you think about
NotificationPlatformBridge? Could do in a follow-up as well..

+ public:
+  static NotificationBridge* Create();
+
+  virtual ~NotificationBridge() {}
+
+  // Shows a toast on screen using the data passed in |notification|.
+  virtual void Display(const std::string& notification_id,
+                       const std::string& profile_id,
+                       bool is_incognito,
+                       const Notification& notification) = 0;
+
+  // Closes a nofication with |notification_id| and |profile_id| if being
+  // displayed.
+  virtual void Close(const std::string& profile_id,
+                     const std::string& notification_id) = 0;
+
+  // Fills in |notifications| with a set of notification ids currently being
+  // displayed for a given profile.
+  // The return value expresses whether the underlying platform has the
+  // capability to provide displayed notifications so the empty set
+  // can be disambiguated.
+  virtual bool GetDisplayed(const std::string& profile_id,
+                            bool incognito,
+                            std::set<std::string>* notification_ids) const = 0;
+
+  // Temporary method while the refactor is finished. It denotes whether
+  // the notifications will be shown by chrome itself or by the OS.
+  // It is needed while migrating MacOSX and Windows to their respective
+  // notification centers since the decision is made at runtime via flags.
+  virtual bool SupportsNotificationCenter() const = 0;
+
+ protected:
+  NotificationBridge() {}
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(NotificationBridge);
+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_BRIDGE_H_