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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilderBase.java

Issue 1656243002: Implementation of renotify flag for Notifications. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added TypeError for incorrect options and unit test Created 4 years, 11 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/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilderBase.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilderBase.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilderBase.java
index 08ef5fb47bd1fc05f5ea60dc671ced168090cdcb..a0668156529b7cd22f53108f22c757ccabab93d1 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilderBase.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilderBase.java
@@ -47,6 +47,7 @@ public abstract class NotificationBuilderBase {
protected Action mSettingsAction;
protected int mDefaults = Notification.DEFAULT_ALL;
protected long[] mVibratePattern;
+ protected boolean mRenotify;
protected long mTimestamp;
/**
@@ -166,6 +167,14 @@ public abstract class NotificationBuilderBase {
}
/**
+ * Sets the behavior for when the notification is replaced.
+ */
+ public NotificationBuilderBase setRenotify(boolean renotify) {
+ mRenotify = renotify;
+ return this;
+ }
+
+ /**
* Sets the timestamp at which the event of the notification took place.
*/
public NotificationBuilderBase setTimestamp(long timestamp) {

Powered by Google App Engine
This is Rietveld 408576698