From 57a777f7584961290e87e6e4149c0ed042334425 Mon Sep 17 00:00:00 2001 From: "rafaelw@chromium.org" Date: Wed, 31 Mar 2010 01:09:42 +0000 Subject: Hook up extension apps notification permission, take two This is the chromium side of a change which will wait to land on the webkit side landing. (https://bugs.webkit.org/show_bug.cgi?id=36625) It changes the NotificationPresenter to pass the sourceURL, rather than the SecurityOrigin in checking permission. The full URL is required to match the app extent. BUG=32361, 31024 TEST=NONE Review URL: http://codereview.chromium.org/1383001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43162 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/renderer/notification_provider.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'chrome/renderer/notification_provider.cc') diff --git a/chrome/renderer/notification_provider.cc b/chrome/renderer/notification_provider.cc index 1bcefee..45fb7b9 100644 --- a/chrome/renderer/notification_provider.cc +++ b/chrome/renderer/notification_provider.cc @@ -52,12 +52,11 @@ void NotificationProvider::objectDestroyed( } WebNotificationPresenter::Permission NotificationProvider::checkPermission( - const WebURL& url, WebDocument* document) { + const WebURL& url) { int permission; Send(new ViewHostMsg_CheckNotificationPermission( view_->routing_id(), url, - document ? UTF16ToASCII(document->applicationID()) : "", &permission)); return static_cast(permission); } -- cgit v1.1