Introduce the Blink NotificationService, move permission checks there

content/browser/notifications/blink_notification_service_impl.h

Index: content/browser/notifications/blink_notification_service_impl.h
diff --git a/content/browser/notifications/blink_notification_service_impl.h b/content/browser/notifications/blink_notification_service_impl.h
new file mode 100644
index 0000000000000000000000000000000000000000..30826de166170ced33ede4273b3362b4924844fd
--- /dev/null
+++ b/content/browser/notifications/blink_notification_service_impl.h
@@ -0,0 +1,53 @@
+// 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 CONTENT_BROWSER_NOTIFICATIONS_BLINK_NOTIFICATION_SERVICE_IMPL_H_
+#define CONTENT_BROWSER_NOTIFICATIONS_BLINK_NOTIFICATION_SERVICE_IMPL_H_
+
+#include "base/macros.h"
+#include "mojo/public/cpp/bindings/binding.h"
+#include "mojo/public/cpp/bindings/interface_request.h"
+#include "third_party/WebKit/public/platform/modules/notifications/notification_service.mojom.h"
+
+namespace content {
+
+class PlatformNotificationContextImpl;
+class ResourceContext;
+
+// Implementation of the NotificationService used for Web Notifications. Is

[ncarter (slow), 2016/05/13 22:04:16]

jam: How confusing will it be to have both blink::mojom::NotificationService and
content/public/browser/notification_service.h? Is there anything we should do,
naming-wise, to reduce possible confusion?

[jam, 2016/05/13 22:16:46]

it's a little confusing I agree. however content::NS is supposed to go away
(https://bugs.chromium.org/p/chromium/issues/detail?id=268984). There aren't
that many remaining, but the work has stalled.

+// responsible for displaying, updating and reading of both non-persistent
+// and persistent notifications. Lives on the IO thread.
+class BlinkNotificationServiceImpl : public blink::mojom::NotificationService {
+ public:
+  BlinkNotificationServiceImpl(
+      PlatformNotificationContextImpl* notification_context,
+      ResourceContext* resource_context,
+      int render_process_id,
+      mojo::InterfaceRequest<blink::mojom::NotificationService> request);
+  ~BlinkNotificationServiceImpl() override;
+
+  // blink::mojom::NotificationService implementation.
+  void GetPermissionStatus(
+      const mojo::String& origin,
+      const GetPermissionStatusCallback& callback) override;
+
+ private:
+  // Called when an error is detected on binding_.
+  void OnConnectionError();
+
+  // The notification context that owns this service instance.
+  PlatformNotificationContextImpl* notification_context_;
+
+  ResourceContext* resource_context_;
+
+  int render_process_id_;
+
+  mojo::Binding<blink::mojom::NotificationService> binding_;
+
+  DISALLOW_COPY_AND_ASSIGN(BlinkNotificationServiceImpl);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_NOTIFICATIONS_BLINK_NOTIFICATION_SERVICE_IMPL_H_