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

Unified Diff: chrome/browser/notifications/notification_permission_context.cc

Issue 2586153002: Move kill switch and secure origin checking logic (Closed)
Patch Set: Move kill switch and secure origin checking logic Created 4 years 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: chrome/browser/notifications/notification_permission_context.cc
diff --git a/chrome/browser/notifications/notification_permission_context.cc b/chrome/browser/notifications/notification_permission_context.cc
index 5f0aa58a6ba843c1cd32dce2092578a358f20542..88a02636281e9c090760fade0176ce7d753bc90c 100644
--- a/chrome/browser/notifications/notification_permission_context.cc
+++ b/chrome/browser/notifications/notification_permission_context.cc
@@ -172,7 +172,7 @@ NotificationPermissionContext::NotificationPermissionContext(
NotificationPermissionContext::~NotificationPermissionContext() {}
-ContentSetting NotificationPermissionContext::GetPermissionStatus(
+ContentSetting NotificationPermissionContext::GetPermissionStatusInternal(
const GURL& requesting_origin,
const GURL& embedding_origin) const {
// Push messaging is only allowed to be granted on top-level origins.
@@ -181,8 +181,8 @@ ContentSetting NotificationPermissionContext::GetPermissionStatus(
return CONTENT_SETTING_BLOCK;
}
- return PermissionContextBase::GetPermissionStatus(requesting_origin,
- embedding_origin);
+ return PermissionContextBase::GetPermissionStatusInternal(requesting_origin,
+ embedding_origin);
}
void NotificationPermissionContext::ResetPermission(
« no previous file with comments | « chrome/browser/notifications/notification_permission_context.h ('k') | chrome/browser/permissions/permission_context_base.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698