| 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 c156dac8824d0aafdbb873e2121e2d7c0439bd1d..b6b5b892e3268e811193f5bdbd6dfd95cd7ed81b 100644
|
| --- a/chrome/browser/notifications/notification_permission_context.cc
|
| +++ b/chrome/browser/notifications/notification_permission_context.cc
|
| @@ -168,9 +168,10 @@ NotificationPermissionContext::NotificationPermissionContext(Profile* profile)
|
| NotificationPermissionContext::~NotificationPermissionContext() {}
|
|
|
| void NotificationPermissionContext::ResetPermission(
|
| - const GURL& requesting_origin,
|
| - const GURL& embedder_origin) {
|
| - DesktopNotificationProfileUtil::ClearSetting(profile(), requesting_origin);
|
| + const url::Origin& requesting_origin,
|
| + const url::Origin& embedder_origin) {
|
| + DesktopNotificationProfileUtil::ClearSetting(
|
| + profile(), GURL(requesting_origin.Serialize()));
|
| }
|
|
|
| void NotificationPermissionContext::CancelPermissionRequest(
|
| @@ -186,8 +187,8 @@ void NotificationPermissionContext::CancelPermissionRequest(
|
| void NotificationPermissionContext::DecidePermission(
|
| content::WebContents* web_contents,
|
| const PermissionRequestID& id,
|
| - const GURL& requesting_origin,
|
| - const GURL& embedding_origin,
|
| + const url::Origin& requesting_origin,
|
| + const url::Origin& embedding_origin,
|
| const BrowserPermissionCallback& callback) {
|
| DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
|
|
|
| @@ -223,18 +224,17 @@ void NotificationPermissionContext::DecidePermission(
|
| // TODO(mukai) Consider why notifications behave differently than
|
| // other permissions. https://crbug.com/416894
|
| void NotificationPermissionContext::UpdateContentSetting(
|
| - const GURL& requesting_origin,
|
| - const GURL& embedder_origin,
|
| + const url::Origin& requesting_origin,
|
| + const url::Origin& embedder_origin,
|
| ContentSetting content_setting) {
|
| DCHECK(content_setting == CONTENT_SETTING_ALLOW ||
|
| content_setting == CONTENT_SETTING_BLOCK);
|
|
|
| + const GURL requesting_url(requesting_origin.Serialize());
|
| if (content_setting == CONTENT_SETTING_ALLOW) {
|
| - DesktopNotificationProfileUtil::GrantPermission(profile(),
|
| - requesting_origin);
|
| + DesktopNotificationProfileUtil::GrantPermission(profile(), requesting_url);
|
| } else {
|
| - DesktopNotificationProfileUtil::DenyPermission(profile(),
|
| - requesting_origin);
|
| + DesktopNotificationProfileUtil::DenyPermission(profile(), requesting_url);
|
| }
|
| }
|
|
|
|
|