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

Unified Diff: chrome/browser/notifications/desktop_notification_service.h

Issue 25492006: Makes the response to CheckNotificationPermission asynchronous. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 2 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/notifications/desktop_notification_service.h
diff --git a/chrome/browser/notifications/desktop_notification_service.h b/chrome/browser/notifications/desktop_notification_service.h
index c77720b4b071fafee049c9179b99816cb3bf53c9..f3ead2611ed2f7ecc19b77318057ee6b8d3f56ea 100644
--- a/chrome/browser/notifications/desktop_notification_service.h
+++ b/chrome/browser/notifications/desktop_notification_service.h
@@ -34,6 +34,10 @@ class WebContents;
struct ShowDesktopNotificationHostMsgParams;
}
+namespace extensions {
+class Extension;
+}
+
namespace gfx {
class Image;
}
@@ -151,8 +155,8 @@ class DesktopNotificationService : public BrowserContextKeyedService,
// Checks to see if a given origin has permission to create desktop
// notifications.
- WebKit::WebNotificationPresenter::Permission
- HasPermission(const GURL& origin);
+ WebKit::WebNotificationPresenter::Permission HasPermission(const GURL& origin,
+ int process_id);
// Returns true if the notifier with |notifier_id| is allowed to send
// notifications.
@@ -194,6 +198,8 @@ class DesktopNotificationService : public BrowserContextKeyedService,
const content::NotificationSource& source,
const content::NotificationDetails& details) OVERRIDE;
+ bool ExtensionApplies(const extensions::Extension* extension);
+
// The profile which owns this object.
Profile* profile_;
« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | chrome/browser/notifications/desktop_notification_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698