diff options
| author | johnnyg@chromium.org <johnnyg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-10 06:27:38 +0000 |
|---|---|---|
| committer | johnnyg@chromium.org <johnnyg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-10 06:27:38 +0000 |
| commit | 9429e976325c93329fd5d35e18699b8c47411f84 (patch) | |
| tree | 4cd2048ae521a351b92d8b50863c52eb97489947 /chrome/renderer/notification_provider.cc | |
| parent | 0c98d920fa52263e940683fea1c8bb2d870f42bb (diff) | |
| download | chromium_src-9429e976325c93329fd5d35e18699b8c47411f84.zip chromium_src-9429e976325c93329fd5d35e18699b8c47411f84.tar.gz chromium_src-9429e976325c93329fd5d35e18699b8c47411f84.tar.bz2 | |
Adds an options menu to the frame of the desktop notification, for now with the single option of revoking permissions from that origin.
The renderer side change is a related issue that the full URL was being sent as the "source" of the notification rather than only the origin.
BUG=26690
TEST=open a notification
Review URL: http://codereview.chromium.org/363011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31547 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/notification_provider.cc')
| -rw-r--r-- | chrome/renderer/notification_provider.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/renderer/notification_provider.cc b/chrome/renderer/notification_provider.cc index 6d142fe..2862e4a 100644 --- a/chrome/renderer/notification_provider.cc +++ b/chrome/renderer/notification_provider.cc @@ -78,7 +78,7 @@ bool NotificationProvider::ShowHTML(const WebNotification& notification, int id) { DCHECK(notification.isHTML()); return Send(new ViewHostMsg_ShowDesktopNotification(view_->routing_id(), - GURL(view_->webview()->mainFrame()->url()), + GURL(view_->webview()->mainFrame()->url()).GetOrigin(), notification.url(), id)); } @@ -86,7 +86,7 @@ bool NotificationProvider::ShowText(const WebNotification& notification, int id) { DCHECK(!notification.isHTML()); return Send(new ViewHostMsg_ShowDesktopNotificationText(view_->routing_id(), - GURL(view_->webview()->mainFrame()->url()), + GURL(view_->webview()->mainFrame()->url()).GetOrigin(), GURL(notification.icon()), notification.title(), notification.body(), id)); } |