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

Unified Diff: chrome/browser/storage/durable_storage_permission_context.cc

Issue 2723983004: Add PermissionManager::GetPermissionStatusForFrame function (Closed)
Patch Set: Add PermissionManager::GetPermissionStatusForFrame function Created 3 years, 9 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: chrome/browser/storage/durable_storage_permission_context.cc
diff --git a/chrome/browser/storage/durable_storage_permission_context.cc b/chrome/browser/storage/durable_storage_permission_context.cc
index c6fd62c4d48f2c1549827f3c5315204ddefb0734..eb31debc8d441c1609f98c6348e9d14c695b09d7 100644
--- a/chrome/browser/storage/durable_storage_permission_context.cc
+++ b/chrome/browser/storage/durable_storage_permission_context.cc
@@ -39,12 +39,14 @@ void DurableStoragePermissionContext::DecidePermission(
bool user_gesture,
const BrowserPermissionCallback& callback) {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
- DCHECK_NE(
- CONTENT_SETTING_ALLOW,
- GetPermissionStatus(requesting_origin, embedding_origin).content_setting);
- DCHECK_NE(
- CONTENT_SETTING_BLOCK,
- GetPermissionStatus(requesting_origin, embedding_origin).content_setting);
+ DCHECK_NE(CONTENT_SETTING_ALLOW,
+ GetPermissionStatus(nullptr /* render_frame_host */,
+ requesting_origin, embedding_origin)
+ .content_setting);
+ DCHECK_NE(CONTENT_SETTING_BLOCK,
+ GetPermissionStatus(nullptr /* render_frame_host */,
+ requesting_origin, embedding_origin)
+ .content_setting);
// Durable is only allowed to be granted to the top-level origin. Embedding
// origin is the last committed navigation origin to the web contents.

Powered by Google App Engine
This is Rietveld 408576698