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

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

Issue 1995663003: Revert of Introduce the Blink NotificationService, move permission checks there (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
deleted file mode 100644
index 8db1cdd25b4935d72f9f15c5d5f60205610478fe..0000000000000000000000000000000000000000
--- a/content/browser/notifications/blink_notification_service_impl.cc
+++ /dev/null
@@ -1,71 +0,0 @@
-// 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