Migrate web notifications to ChromeNotificationBuilder

chrome/android/java/src/org/chromium/chrome/browser/notifications/StandardNotificationBuilder.java

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..84b226ec91180b36802f41e83b54352d91e2b55f 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,14 @@ 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)

[nyquist, 2017/02/28 07:29:09]

This seems unsafe. Could you use mContext.getApplicationContext() to ensure it
is in fact the application context here?

[awdf, 2017/02/28 18:04:11]

Done. (It should already be as NPB gets application context which is passed
through, but agree this is more robust)

+                        .createChromeNotificationBuilder(false /* preferCompat */,
+                                NotificationConstants.CATEGORY_ID_GENERAL,
+                                mContext.getString(org.chromium.chrome.R.string
+                                                           .notification_category_general));
+
         builder.setContentTitle(mTitle);
         builder.setContentText(mBody);
         builder.setSubText(mOrigin);