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

Unified Diff: third_party/WebKit/Source/modules/notifications/NotificationManager.cpp

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: third_party/WebKit/Source/modules/notifications/NotificationManager.cpp
diff --git a/third_party/WebKit/Source/modules/notifications/NotificationManager.cpp b/third_party/WebKit/Source/modules/notifications/NotificationManager.cpp
deleted file mode 100644
index 9bda0a1f198ef6ca8c2c08a2ae129fa87927987b..0000000000000000000000000000000000000000
--- a/third_party/WebKit/Source/modules/notifications/NotificationManager.cpp
+++ /dev/null
@@ -1,67 +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 "modules/notifications/NotificationManager.h"
-
-#include "platform/weborigin/SecurityOrigin.h"
-#include "public/platform/Platform.h"
-#include "public/platform/ServiceRegistry.h"
-#include "public/platform/modules/permissions/permission_status.mojom-blink.h"
-
-namespace blink {
-
-// static
-NotificationManager* NotificationManager::from(ExecutionContext* executionContext)
-{
- DCHECK(executionContext);
- DCHECK(executionContext->isContextThread());
-
- NotificationManager* manager = static_cast<NotificationManager*>(Supplement<ExecutionContext>::from(executionContext, supplementName()));
- if (!manager) {
- manager = new NotificationManager(executionContext);
- Supplement<ExecutionContext>::provideTo(*executionContext, supplementName(), manager);
- }
-
- return manager;
-}
-
-// static
-const char* NotificationManager::supplementName()
-{
- return "NotificationManager";
-}
-
-NotificationManager::NotificationManager(ExecutionContext* executionContext)
- : ContextLifecycleObserver(executionContext)
-{
- Platform::current()->serviceRegistry()->connectToRemoteService(mojo::GetProxy(&m_service));
-}
-
-NotificationManager::~NotificationManager()
-{
-}
-
-mojom::blink::PermissionStatus NotificationManager::permissionStatus() const
-{
- mojom::blink::PermissionStatus permissionStatus;
-
- const bool result =
- m_service->GetPermissionStatus(getExecutionContext()->getSecurityOrigin()->toString(), &permissionStatus);
- DCHECK(result);
-
- return permissionStatus;
-}
-
-void NotificationManager::contextDestroyed()
-{
- m_service.reset();
-}
-
-DEFINE_TRACE(NotificationManager)
-{
- ContextLifecycleObserver::trace(visitor);
- Supplement<ExecutionContext>::trace(visitor);
-}
-
-} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698