Notifications: change some origin parameters from string to GURL and
get rid of notification_id in places where it's redundant with the
notification struct.
Bug: none
Change-Id: I9bc9449e5e75c3c3b9eed46bb6572664e11bdc68
Reviewed-on: https://chromium-review.googlesource.com/750107
Commit-Queue: Evan Stade <[email protected]>
Reviewed-by: Scott Violet <[email protected]>
Reviewed-by: Justin DeWitt <[email protected]>
Reviewed-by: Steven Bennetts <[email protected]>
Cr-Commit-Position: refs/heads/master@{#513926}diff --git a/chrome/browser/notifications/non_persistent_notification_handler.h b/chrome/browser/notifications/non_persistent_notification_handler.h
index b38cc1e..a7f5534f 100644
--- a/chrome/browser/notifications/non_persistent_notification_handler.h
+++ b/chrome/browser/notifications/non_persistent_notification_handler.h
@@ -17,12 +17,12 @@
// NotificationHandler implementation
void OnShow(Profile* profile, const std::string& notification_id) override;
void OnClose(Profile* profile,
- const std::string& origin,
+ const GURL& origin,
const std::string& notification_id,
bool by_user) override;
void OnClick(Profile* profile,
- const std::string& origin,
+ const GURL& origin,
const std::string& notification_id,
const base::Optional<int>& action_index,
const base::Optional<base::string16>& reply) override;
