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

Unified Diff: chrome/browser/push_messaging/push_messaging_service_impl.cc

Issue 1093923005: Swap hasPermission with permissionState in browser_tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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: chrome/browser/push_messaging/push_messaging_service_impl.cc
diff --git a/chrome/browser/push_messaging/push_messaging_service_impl.cc b/chrome/browser/push_messaging/push_messaging_service_impl.cc
index 68ec65ffc1da48aeb021bf638d95db58c548c8a8..11907fe0aef82fdc604ad56ccffb5078648836e8 100644
--- a/chrome/browser/push_messaging/push_messaging_service_impl.cc
+++ b/chrome/browser/push_messaging/push_messaging_service_impl.cc
@@ -83,7 +83,7 @@ blink::WebPushPermissionStatus ToPushPermission(ContentSetting setting) {
case CONTENT_SETTING_BLOCK:
return blink::WebPushPermissionStatusDenied;
case CONTENT_SETTING_ASK:
- return blink::WebPushPermissionStatusDefault;
+ return blink::WebPushPermissionStatusPrompt;
default:
NOTREACHED();
return blink::WebPushPermissionStatusDenied;

Powered by Google App Engine
This is Rietveld 408576698