[NOT FOR REVIEW] Significantly simplify the PlatformNotificationService //content API.

content/public/browser/platform_notification_service.h

 // Copyright 2014 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 CONTENT_PUBLIC_BROWSER_PLATFORM_NOTIFICATION_SERVICE_H_
 #define CONTENT_PUBLIC_BROWSER_PLATFORM_NOTIFICATION_SERVICE_H_
 
-#include <stdint.h>
 #include <string>
 
-#include "base/callback_forward.h"
-#include "base/memory/scoped_ptr.h"
 #include "content/common/content_export.h"
 #include "third_party/WebKit/public/platform/modules/notifications/WebNotificationPermission.h"
 
 class GURL;
-class SkBitmap;
+
+namespace message_center {
+class Notification;
+}
 
 namespace content {
 
 class BrowserContext;
-class DesktopNotificationDelegate;
-struct PlatformNotificationData;
 class ResourceContext;
 
-// The service using which notifications can be presented to the user. There
-// should be a unique instance of the PlatformNotificationService depending
-// on the browsing context being used.
+// Interface for the service through which platform notifications can be
+// presented to the user.
 class CONTENT_EXPORT PlatformNotificationService {
  public:
   virtual ~PlatformNotificationService() {}
 
   // Checks if |origin| has permission to display Web Notifications.
   // This method must only be called on the UI thread.
   virtual blink::WebNotificationPermission CheckPermissionOnUIThread(
       BrowserContext* browser_context,
       const GURL& origin,
       int render_process_id) = 0;
 
   // Checks if |origin| has permission to display Web Notifications. This method
   // exists to serve the synchronous IPC required by the Notification.permission
   // JavaScript getter, and should not be used for other purposes. See
   // https://crbug.com/446497 for the plan to deprecate this method.
   // This method must only be called on the IO thread.
   virtual blink::WebNotificationPermission CheckPermissionOnIOThread(
       ResourceContext* resource_context,
       const GURL& origin,
       int render_process_id) = 0;
 
-  // Displays the notification described in |params| to the user. A closure
-  // through which the notification can be closed will be stored in the
-  // |cancel_callback| argument. This method must be called on the UI thread.
+  // Displays the |notification| to the user. This method must be called on the

[dewittj, 2015/05/02 18:43:29]

I like the simplification.  If layering is an issue we can always use another
struct besides m_c::Notification and teach notifications how to use them.

+  // UI thread.
   virtual void DisplayNotification(
       BrowserContext* browser_context,
-      const GURL& origin,
-      const SkBitmap& icon,
-      const PlatformNotificationData& notification_data,
-      scoped_ptr<DesktopNotificationDelegate> delegate,
-      base::Closure* cancel_callback) = 0;
+      const message_center::Notification& notification) = 0;
 
-  // Displays the persistent notification described in |notification_data| to
-  // the user. This method must be called on the UI thread.
-  virtual void DisplayPersistentNotification(
-      BrowserContext* browser_context,
-      int64_t persistent_notification_id,
-      const GURL& origin,
-      const SkBitmap& icon,
-      const PlatformNotificationData& notification_data) = 0;
-
-  // Closes the persistent notification identified by
-  // |persistent_notification_id|. This method must be called on the UI thread.
-  virtual void ClosePersistentNotification(
-      BrowserContext* browser_context,
-      int64_t persistent_notification_id) = 0;
+  // Ensures that the notification identified by |notification_id| will be
+  // closed for the user. This method must be called on the UI thread.
+  virtual void CloseNotification(BrowserContext* browser_context,
+                                 const std::string& notification_id) = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_PLATFORM_NOTIFICATION_SERVICE_H_