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

Unified Diff: ui/message_center/popup_timer.cc

Issue 1395093002: Fix system notifications incorrectly marked as type WEB_PAGE (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@timeout
Patch Set: Created 5 years, 2 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: ui/message_center/popup_timer.cc
diff --git a/ui/message_center/popup_timer.cc b/ui/message_center/popup_timer.cc
index f2f9c5e015bbde89e53dbb257cd6a1f65028a7a8..a8971ee1be834e01fbe686b1e8ad8f6a780e421e 100644
--- a/ui/message_center/popup_timer.cc
+++ b/ui/message_center/popup_timer.cc
@@ -16,9 +16,9 @@ namespace message_center {
namespace {
base::TimeDelta GetTimeoutForNotification(Notification* notification) {
- if (notification->priority() > message_center::DEFAULT_PRIORITY)
+ if (notification->priority() > DEFAULT_PRIORITY)
return base::TimeDelta::FromSeconds(kAutocloseHighPriorityDelaySeconds);
- if (notification->is_web_notification())
+ if (notification->notifier_id().type == NotifierId::WEB_PAGE)
return base::TimeDelta::FromSeconds(kAutocloseWebNotificationDelaySeconds);
Peter Beverloo 2015/10/09 10:47:30 nit: this would now be more appropriately named as
johnme 2015/10/09 11:21:02 Done.
return base::TimeDelta::FromSeconds(kAutocloseDefaultDelaySeconds);
}

Powered by Google App Engine
This is Rietveld 408576698