From 95cec243879c02b01a83bf654361ccf398fab42a Mon Sep 17 00:00:00 2001 From: Jeremy Apthorp Date: Tue, 2 Oct 2018 13:52:21 -0700 Subject: [PATCH] FIXME: Move persistent notification id from notification database to profile https://chromium-review.googlesource.com/c/chromium/src/+/1114840 --- brightray/browser/platform_notification_service.cc | 6 ++++++ brightray/browser/platform_notification_service.h | 2 ++ 2 files changed, 8 insertions(+) diff --git a/brightray/browser/platform_notification_service.cc b/brightray/browser/platform_notification_service.cc index 630dbe0ad02c..cb653e3ddcfb 100644 --- a/brightray/browser/platform_notification_service.cc +++ b/brightray/browser/platform_notification_service.cc @@ -121,4 +121,10 @@ void PlatformNotificationService::GetDisplayedNotifications( content::BrowserContext* browser_context, const DisplayedNotificationsCallback& callback) {} +int64_t PlatformNotificationService::ReadNextPersistentNotificationId( + content::BrowserContext* browser_context) { + // Electron doesn't support persistent notifications. + return 0; +} + } // namespace brightray diff --git a/brightray/browser/platform_notification_service.h b/brightray/browser/platform_notification_service.h index 7043a88aa509..6ad8d9cf6dcf 100644 --- a/brightray/browser/platform_notification_service.h +++ b/brightray/browser/platform_notification_service.h @@ -43,6 +43,8 @@ class PlatformNotificationService void GetDisplayedNotifications( content::BrowserContext* browser_context, const DisplayedNotificationsCallback& callback) override; + int64_t ReadNextPersistentNotificationId( + content::BrowserContext* browser_context) override; private: BrowserClient* browser_client_;