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

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

Issue 2686463002: Add a source to the result of PermissionContextBase::GetPermissionStatus (Closed)
Patch Set: Add a source to the result of PermissionContextBase::GetPermissionStatus Created 3 years, 10 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 15bc3c66508e90df27e13a182b9e1d9f016c0371..97cf41c70d6819d4b895dc3a5cb38314abde27ca 100644
--- a/chrome/browser/storage/durable_storage_permission_context.cc
+++ b/chrome/browser/storage/durable_storage_permission_context.cc
@@ -41,10 +41,12 @@ 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));
- DCHECK_NE(CONTENT_SETTING_BLOCK,
- GetPermissionStatus(requesting_origin, embedding_origin));
+ DCHECK_NE(
+ CONTENT_SETTING_ALLOW,
+ GetPermissionStatus(requesting_origin, embedding_origin).content_setting);
+ DCHECK_NE(
+ CONTENT_SETTING_BLOCK,
+ GetPermissionStatus(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