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"

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

micro nit: don't we usually have blank lines between STL includes and
Chrome-based ones?

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

Done.

+
+class Notification;
+
+// This class bridges the display and click operations available on a
+// notification.

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

Let's try to be a bit more inclusive. What about:

// 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.

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

Except that right now there is no bridge for chrome notifications but ok.

+// TODO(miguelg): Add support for click and close events.
+class NotificationBridge {
+ public:
+  static NotificationBridge* Create();
+
+  NotificationBridge() {}

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

If we need a static Create method, let's make the constructor protected. A
hypothetical FakeNotificationBridge could increase visibility as needed.

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

Done.

+  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.

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

Please do document what it does and why it's significant.

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

Done.

+  virtual bool SupportsNotificationCenter() const = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(NotificationBridge);
+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_BRIDGE_H_