Migrate web notifications to ChromeNotificationBuilder

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

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.notifications;
 
 import android.app.Notification;
 import android.app.PendingIntent;
 import android.content.Context;
 import android.graphics.Bitmap;
+import android.graphics.drawable.Icon;
 import android.support.v4.app.NotificationCompat;
+import android.widget.RemoteViews;
 
 /**
  * Wraps a NotificationCompat.Builder object.
  * Created by awdf on 2/16/17.
  */
 public class NotificationCompatBuilder implements ChromeNotificationBuilder {
     private final NotificationCompat.Builder mBuilder;
 
     public NotificationCompatBuilder(Context context) {
         mBuilder = new NotificationCompat.Builder(context);
     }
 
     @Override
     public ChromeNotificationBuilder setAutoCancel(boolean autoCancel) {
         mBuilder.setAutoCancel(autoCancel);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setContentIntent(PendingIntent contentIntent) {
         mBuilder.setContentIntent(contentIntent);
         return this;
     }
 
     @Override
-    public ChromeNotificationBuilder setContentTitle(String title) {
+    public ChromeNotificationBuilder setContentTitle(CharSequence title) {
         mBuilder.setContentTitle(title);
         return this;
     }
 
     @Override
-    public ChromeNotificationBuilder setContentText(String text) {
+    public ChromeNotificationBuilder setContentText(CharSequence text) {
         mBuilder.setContentText(text);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setSmallIcon(int icon) {
         mBuilder.setSmallIcon(icon);
         return this;
     }
 
     @Override
-    public ChromeNotificationBuilder setTicker(String text) {
+    public ChromeNotificationBuilder setSmallIcon(Icon icon) {
+        return this;
+    }
+
+    @Override
+    public ChromeNotificationBuilder setTicker(CharSequence text) {
         mBuilder.setTicker(text);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setLocalOnly(boolean localOnly) {
         mBuilder.setLocalOnly(localOnly);
         return this;
     }
 
@@ skipping 15 matching lines @@
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setShowWhen(boolean showWhen) {
         mBuilder.setShowWhen(showWhen);
         return this;
     }
 
     @Override
-    public ChromeNotificationBuilder addAction(int icon, String title, PendingIntent intent) {
+    public ChromeNotificationBuilder addAction(int icon, CharSequence title, PendingIntent intent) {
         mBuilder.addAction(icon, title, intent);
         return this;
     }
 
     @Override
+    public ChromeNotificationBuilder addAction(Notification.Action action) {
+        return this;
+    }
+
+    @Override
     public ChromeNotificationBuilder setDeleteIntent(PendingIntent intent) {
         mBuilder.setDeleteIntent(intent);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setPriority(int pri) {
         mBuilder.setPriority(pri);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setProgress(int max, int percentage, boolean indeterminate) {
         mBuilder.setProgress(max, percentage, indeterminate);
         return this;
     }
 
     @Override
-    public ChromeNotificationBuilder setSubText(String text) {
+    public ChromeNotificationBuilder setSubText(CharSequence text) {
         mBuilder.setSubText(text);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setContentInfo(String info) {
         mBuilder.setContentInfo(info);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setWhen(long time) {
         mBuilder.setWhen(time);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setLargeIcon(Bitmap icon) {
         mBuilder.setLargeIcon(icon);
         return this;
     }
 
     @Override
     public ChromeNotificationBuilder setVibrate(long[] vibratePattern) {
         mBuilder.setVibrate(vibratePattern);
         return this;
     }
 
     @Override
+    public ChromeNotificationBuilder setDefaults(int defaults) {
+        mBuilder.setDefaults(defaults);
+        return this;
+    }
+
+    public ChromeNotificationBuilder setOnlyAlertOnce(boolean onlyAlertOnce) {
+        mBuilder.setOnlyAlertOnce(onlyAlertOnce);
+        return this;
+    }
+
+    @Override
+    public ChromeNotificationBuilder setPublicVersion(Notification publicNotification) {
+        mBuilder.setPublicVersion(publicNotification);
+        return this;
+    }
+
+    @Override
+    public ChromeNotificationBuilder setContent(RemoteViews views) {
+        mBuilder.setCustomContentView(views);
+        return this;
+    }
+
+    @Override
+    public ChromeNotificationBuilder setStyle(Notification.BigPictureStyle style) {
+        return this;

[Peter Beverloo, 2017/02/20 01:10:20]

Maybe "assert false;  // unused" or something to clarify that this should have
no callers?

[awdf, 2017/02/24 17:37:30]

Done, and in the other places it where it no-ops.

+    }
+
+    @Override
+    public ChromeNotificationBuilder setStyle(Notification.BigTextStyle bigTextStyle) {
+        return this;
+    }
+
+    @Override
+    public Notification buildWithBigContentView(RemoteViews view) {
+        return mBuilder.setCustomBigContentView(view).build();
+    }
+
+    @Override
     public Notification buildWithBigTextStyle(String bigText) {
         NotificationCompat.BigTextStyle bigTextStyle =
                 new NotificationCompat.BigTextStyle(mBuilder);
         bigTextStyle.bigText(bigText);
         return bigTextStyle.build();
     }
 
     @Override
     public Notification build() {
         return mBuilder.build();
     }
 }