Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/notifications/StandardNotificationBuilder.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/StandardNotificationBuilder.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/StandardNotificationBuilder.java |
| index 76460297090d0211f6b33146d4a734ec70849a82..478ac3f80d2ecb93b6f350668d3c733bf4f1a4cb 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/StandardNotificationBuilder.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/notifications/StandardNotificationBuilder.java |
| @@ -8,6 +8,8 @@ import android.app.Notification; |
| import android.content.Context; |
| import android.os.Build; |
| +import org.chromium.chrome.browser.ChromeApplication; |
| + |
| /** |
| * Builds a notification using the standard Notification.BigTextStyle layout. |
| */ |
| @@ -23,7 +25,11 @@ public class StandardNotificationBuilder extends NotificationBuilderBase { |
| public Notification build() { |
| // Note: 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(awdf) We should probably use a Compat builder. |
| + ChromeNotificationBuilder builder = |
| + ((ChromeApplication) mContext) |
| + .getChromeNotificationBuilder(false, NotificationConstants.TYPE_ID_GENERAL, |
| + NotificationConstants.TYPE_NAME_GENERAL); |
|
Peter Beverloo
2017/02/20 01:10:20
dito re: types
|
| builder.setContentTitle(mTitle); |
| builder.setContentText(mBody); |
| builder.setSubText(mOrigin); |