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

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

Issue 1116693002: [NOT FOR REVIEW] Significantly simplify the PlatformNotificationService //content API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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..9eefee08caf42a0555773ebb39b74dc9910fbb54 100644
--- a/content/browser/notifications/page_notification_delegate.h
+++ b/content/browser/notifications/page_notification_delegate.h
@@ -5,24 +5,27 @@
#ifndef CONTENT_BROWSER_NOTIFICATIONS_PAGE_NOTIFICATION_DELEGATE_H_
#define CONTENT_BROWSER_NOTIFICATIONS_PAGE_NOTIFICATION_DELEGATE_H_
-#include "content/public/browser/desktop_notification_delegate.h"
+#include "ui/message_center/notification_delegate.h"
namespace content {
// A delegate used by the notification service to report the results of platform
// notification interactions to Web Notifications whose lifetime is tied to
// that of the page displaying them.
-class PageNotificationDelegate : public DesktopNotificationDelegate {
+class PageNotificationDelegate : public message_center::NotificationDelegate {
public:
PageNotificationDelegate(int render_process_id, int notification_id);
- ~PageNotificationDelegate() override;
- // DesktopNotificationDelegate implementation.
- void NotificationDisplayed() override;
- void NotificationClosed() override;
- void NotificationClick() override;
+ // message_center::NotificationDelegate implementation.
+ void Display() override;
+ void Close(bool by_user) override;
+ bool HasClickedListener() override;
+ void Click() override;
+ void ButtonClick(int button_index) override {}
private:
+ ~PageNotificationDelegate() override;
+
int render_process_id_;
int notification_id_;
};

Powered by Google App Engine
This is Rietveld 408576698