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

Unified Diff: content/browser/notifications/page_notification_delegate.h

Issue 1904163002: Move Web Notifications to use Mojo Base URL: https://chromium.googlesource.com/chromium/src.git@skbitmap-blink
Patch Set: it works \o/ Created 4 years, 8 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: content/browser/notifications/page_notification_delegate.h
diff --git a/content/browser/notifications/page_notification_delegate.h b/content/browser/notifications/page_notification_delegate.h
index fd45f9152bc25b5e20eb3cab6c53afa77ab71a05..b3439158f535402f5ab85f124d32513fa888cf8e 100644
--- a/content/browser/notifications/page_notification_delegate.h
+++ b/content/browser/notifications/page_notification_delegate.h
@@ -6,6 +6,7 @@
#define CONTENT_BROWSER_NOTIFICATIONS_PAGE_NOTIFICATION_DELEGATE_H_
#include "content/public/browser/desktop_notification_delegate.h"
+#include "third_party/WebKit/public/platform/modules/notifications/notification_service.mojom.h"
namespace content {
@@ -14,7 +15,8 @@ namespace content {
// that of the page displaying them.
class PageNotificationDelegate : public DesktopNotificationDelegate {
public:
- PageNotificationDelegate(int render_process_id, int notification_id);
+ explicit PageNotificationDelegate(
+ blink::mojom::NotificationClientPtr notification_client);
~PageNotificationDelegate() override;
// DesktopNotificationDelegate implementation.
@@ -23,8 +25,8 @@ class PageNotificationDelegate : public DesktopNotificationDelegate {
void NotificationClick() override;
private:
- int render_process_id_;
- int notification_id_;
+ // The client to which events will be dispatched.
+ blink::mojom::NotificationClientPtr notification_client_;
};
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698