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

Unified Diff: chrome/browser/notifications/notification_platform_bridge_linux.cc

Issue 2866363002: Linux native notifications: Support infinite timeouts (Closed)
Patch Set: add unit test Created 3 years, 7 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: chrome/browser/notifications/notification_platform_bridge_linux.cc
diff --git a/chrome/browser/notifications/notification_platform_bridge_linux.cc b/chrome/browser/notifications/notification_platform_bridge_linux.cc
index 1a05a57209b038e351ca94cdf11eac7b0d41fd46..6bcf7a2f34ecdf5bcddd159f33669ebb6a253429 100644
--- a/chrome/browser/notifications/notification_platform_bridge_linux.cc
+++ b/chrome/browser/notifications/notification_platform_bridge_linux.cc
@@ -538,7 +538,9 @@ class NotificationPlatformBridgeLinuxImpl
writer.CloseContainer(&hints_writer);
const int32_t kExpireTimeoutDefault = -1;
- writer.AppendInt32(kExpireTimeoutDefault);
+ const int32_t kExpireTimeoutNever = 0;
+ writer.AppendInt32(notification->never_timeout() ? kExpireTimeoutNever
+ : kExpireTimeoutDefault);
std::unique_ptr<dbus::Response> response =
notification_proxy_->CallMethodAndBlock(

Powered by Google App Engine
This is Rietveld 408576698