Index: chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java |
index 94b138c15da4fdd46fd7dbb36414f410aa69f5d6..1e2a414db26d5d5253f9b677b8144081ea07a23c 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java |
@@ -24,6 +24,7 @@ import org.chromium.base.ApiCompatibilityUtils; |
import org.chromium.base.VisibleForTesting; |
import org.chromium.base.metrics.RecordHistogram; |
import org.chromium.chrome.R; |
+import org.chromium.chrome.browser.ChromeApplication; |
import org.chromium.ui.base.LocalizationUtils; |
import java.util.Date; |
@@ -137,9 +138,18 @@ public class CustomNotificationBuilder extends NotificationBuilderBase { |
addActionButtons(bigView); |
configureSettingsButton(bigView); |
- // Note: this is not a NotificationCompat builder so be mindful of the |
+ // Note: under the hood this is not a NotificationCompat builder so be mindful of the |
// API level of methods you call on the builder. |
- Notification.Builder builder = new Notification.Builder(mContext); |
+ // TODO(crbug.com/697104) We should probably use a Compat builder. |
+ ChromeNotificationBuilder builder = |
+ ((ChromeApplication) mContext.getApplicationContext()) |
+ .createChromeNotificationBuilder(false /* preferCompat */, |
+ NotificationConstants.CATEGORY_ID_SITES, |
+ mContext.getString( |
+ org.chromium.chrome.R.string.notification_category_sites), |
+ NotificationConstants.CATEGORY_GROUP_ID_GENERAL, |
+ mContext.getString(org.chromium.chrome.R.string |
+ .notification_category_group_general)); |
builder.setTicker(mTickerText); |
builder.setContentIntent(mContentIntent); |
builder.setDeleteIntent(mDeleteIntent); |
@@ -147,7 +157,7 @@ public class CustomNotificationBuilder extends NotificationBuilderBase { |
builder.setVibrate(mVibratePattern); |
builder.setWhen(mTimestamp); |
builder.setOnlyAlertOnce(!mRenotify); |
- ApiCompatibilityUtils.setContentViewForNotificationBuilder(builder, compactView); |
+ builder.setContent(compactView); |
// Some things are duplicated in the builder to ensure the notification shows correctly on |
// Wear devices and custom lock screens. |
@@ -163,12 +173,9 @@ public class CustomNotificationBuilder extends NotificationBuilderBase { |
addActionToBuilder(builder, mSettingsAction); |
} |
setGroupOnBuilder(builder, mOrigin); |
- if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) { |
- // Notification.Builder.setPublicVersion was added in Android L. |
- builder.setPublicVersion(createPublicNotification(mContext)); |
- } |
+ builder.setPublicVersion(createPublicNotification(mContext)); |
- return ApiCompatibilityUtils.notificationWithBigContentView(builder, bigView); |
+ return builder.buildWithBigContentView(bigView); |
} |
/** |