Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(406)

Unified Diff: content/browser/notifications/blink_notification_service_impl.cc

Issue 2003023002: Introduce the Blink NotificationService, move permission checks there (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/notifications/blink_notification_service_impl.cc
diff --git a/content/browser/notifications/blink_notification_service_impl.cc b/content/browser/notifications/blink_notification_service_impl.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8db1cdd25b4935d72f9f15c5d5f60205610478fe
--- /dev/null
+++ b/content/browser/notifications/blink_notification_service_impl.cc
@@ -0,0 +1,71 @@
+// 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.
+
+#include "content/browser/notifications/blink_notification_service_impl.h"
+
+#include "base/logging.h"
+#include "content/browser/notifications/platform_notification_context_impl.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/content_browser_client.h"
+#include "content/public/browser/platform_notification_service.h"
+#include "content/public/common/content_client.h"
+#include "third_party/WebKit/public/platform/modules/permissions/permission_status.mojom.h"
+#include "url/gurl.h"
+
+namespace content {
+
+namespace {
+
+// Returns the implementation of the PlatformNotificationService. May be NULL.
+PlatformNotificationService* Service() {
+ return GetContentClient()->browser()->GetPlatformNotificationService();
+}
+
+} // namespace
+
+BlinkNotificationServiceImpl::BlinkNotificationServiceImpl(
+ PlatformNotificationContextImpl* notification_context,
+ ResourceContext* resource_context,
+ int render_process_id,
+ mojo::InterfaceRequest<blink::mojom::NotificationService> request)
+ : notification_context_(notification_context),
+ resource_context_(resource_context),
+ render_process_id_(render_process_id),
+ binding_(this, std::move(request)) {
+ DCHECK_CURRENTLY_ON(BrowserThread::IO);
+ DCHECK(notification_context_);
+ DCHECK(resource_context_);
+
+ binding_.set_connection_error_handler(
+ base::Bind(&BlinkNotificationServiceImpl::OnConnectionError,
+ base::Unretained(this) /* the channel is owned by this */));
+}
+
+BlinkNotificationServiceImpl::~BlinkNotificationServiceImpl() {
+ DCHECK_CURRENTLY_ON(BrowserThread::IO);
+}
+
+void BlinkNotificationServiceImpl::GetPermissionStatus(
+ const mojo::String& origin,
+ const GetPermissionStatusCallback& callback) {
+ DCHECK_CURRENTLY_ON(BrowserThread::IO);
+
+ if (!Service()) {
+ callback.Run(blink::mojom::PermissionStatus::DENIED);
+ return;
+ }
+
+ blink::mojom::PermissionStatus permission_status =
+ Service()->CheckPermissionOnIOThread(
+ resource_context_, GURL(origin.get()), render_process_id_);
+
+ callback.Run(permission_status);
+}
+
+void BlinkNotificationServiceImpl::OnConnectionError() {
+ notification_context_->RemoveService(this);
+ // |this| has now been deleted.
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698