Make PlatformNotificationContext display all notifications.
This hands off the display call to PlatformNotificationContext which
gets rid of some code duplication as we always want to show a
notification after writing it into the NotificationDatabase.
Bug: 891339
Change-Id: If0787fcd9668cb0e419edcc16a56c91649983dcc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1511416
Reviewed-by: Avi Drissman <[email protected]>
Reviewed-by: Peter Beverloo <[email protected]>
Commit-Queue: Richard Knoll <[email protected]>
Cr-Commit-Position: refs/heads/master@{#643370}diff --git a/chrome/browser/push_messaging/push_messaging_notification_manager.h b/chrome/browser/push_messaging/push_messaging_notification_manager.h
index 626d82a..fe054ef0 100644
--- a/chrome/browser/push_messaging/push_messaging_notification_manager.h
+++ b/chrome/browser/push_messaging/push_messaging_notification_manager.h
@@ -25,11 +25,7 @@
namespace content {
struct NotificationDatabaseData;
class WebContents;
-}
-
-namespace blink {
-struct PlatformNotificationData;
-}
+} // namespace content
// Developers may be required to display a Web Notification in response to an
// incoming push message in order to clarify to the user that something has
@@ -82,12 +78,9 @@
base::OnceClosure message_handled_closure,
bool silent_push_allowed);
- void DidWriteNotificationData(
- const GURL& origin,
- const blink::PlatformNotificationData& notification_data,
- base::OnceClosure message_handled_closure,
- bool success,
- const std::string& notification_id);
+ void DidWriteNotificationData(base::OnceClosure message_handled_closure,
+ bool success,
+ const std::string& notification_id);
#if defined(OS_CHROMEOS)
bool ShouldSkipUserVisibleOnlyRequirements(const GURL& origin);
