Extract NotificationBuilderBase for holding the arguments.

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

 // Copyright 2015 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.content.res.Resources;
 import android.graphics.Bitmap;
 import android.graphics.BitmapFactory;
 import android.graphics.drawable.BitmapDrawable;
 import android.graphics.drawable.Drawable;
 import android.os.Build;
 import android.support.v4.app.NotificationCompat;
 import android.support.v4.app.NotificationCompat.Action;
 import android.text.format.DateFormat;
 import android.util.DisplayMetrics;
 import android.util.TypedValue;
 import android.view.View;
 import android.widget.RemoteViews;
 
 import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.base.VisibleForTesting;
 import org.chromium.chrome.R;
 import org.chromium.ui.base.LocalizationUtils;
 
-import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.Date;
-import java.util.List;
-
-import javax.annotation.Nullable;
 
 /**
  * Builds a notification using the given inputs. Uses RemoteViews to provide a custom layout.
  */
-public class CustomNotificationBuilder implements NotificationBuilder {
-    /**
-     * Maximum length of CharSequence inputs to prevent excessive memory consumption. At current
-     * screen sizes we display about 500 characters at most, so this is a pretty generous limit, and
-     * it matches what NotificationCompat does.
-     */
-    @VisibleForTesting static final int MAX_CHARSEQUENCE_LENGTH = 5 * 1024;
-
-    /**
-     * The maximum number of action buttons. One is for the settings button, and two more slots are
-     * for developer provided buttons.
-     */
-    private static final int MAX_ACTION_BUTTONS = 3;
-
+public class CustomNotificationBuilder extends NotificationBuilderBase {
     /**
      * The maximum number of lines of body text for the expanded state. Fewer lines are used when
      * the text is scaled up, with a minimum of one line.
      */
     private static final int MAX_BODY_LINES = 7;
 
     /**
      * The fontScale considered large for the purposes of layout.
      */
     private static final float FONT_SCALE_LARGE = 1.3f;
@@ skipping 20 matching lines @@
      */
     private static final int WORK_PROFILE_BADGE_SIZE_DP = 16;
 
     /**
      * Material Grey 600 - to be applied to action button icons in the Material theme.
      */
     private static final int BUTTON_ICON_COLOR_MATERIAL = 0xff757575;
 
     private final Context mContext;
 
-    private CharSequence mTitle;
-    private CharSequence mBody;
-    private CharSequence mOrigin;
-    private CharSequence mTickerText;
-    private Bitmap mLargeIcon;
-    private int mSmallIconId;
-    private PendingIntent mContentIntent;
-    private PendingIntent mDeleteIntent;
-    private List<Action> mActions = new ArrayList<>(MAX_ACTION_BUTTONS);
-    private Action mSettingsAction;
-    private int mDefaults = Notification.DEFAULT_ALL;
-    private long[] mVibratePattern;
-
     public CustomNotificationBuilder(Context context) {
         mContext = context;
     }
 
     @Override
     public Notification build() {
-        // TODO(mvanouwerkerk): Try inheriting from StandardNotificationBuilder to reduce
-        // duplication.
-
         // A note about RemoteViews and updating notifications. When a notification is passed to the
         // {@code NotificationManager} with the same tag and id as a previous notification, an
         // in-place update will be performed. In that case, the actions of all new
         // {@link RemoteViews} will be applied to the views of the old notification. This is safe
         // for actions that overwrite old values such as setting the text of a {@code TextView}, but
         // care must be taken for additive actions. Especially in the case of
         // {@link RemoteViews#addView} the result could be to append new views below stale ones. In
         // that case {@link RemoteViews#removeAllViews} must be called before adding new ones.
         RemoteViews compactView =
                 new RemoteViews(mContext.getPackageName(), R.layout.web_notification);
@@ skipping 46 matching lines @@
         }
         if (mSettingsAction != null) {
             builder.addAction(mSettingsAction);
         }
 
         Notification notification = builder.build();
         notification.bigContentView = bigView;
         return notification;
     }
 
-    @Override
-    public NotificationBuilder setTitle(CharSequence title) {
-        mTitle = limitLength(title);
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setBody(CharSequence body) {
-        mBody = limitLength(body);
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setOrigin(CharSequence origin) {
-        mOrigin = limitLength(origin);
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setTicker(CharSequence tickerText) {
-        mTickerText = limitLength(tickerText);
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setLargeIcon(Bitmap icon) {
-        mLargeIcon = icon;
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setSmallIcon(int iconId) {
-        mSmallIconId = iconId;
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setContentIntent(PendingIntent intent) {
-        mContentIntent = intent;
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setDeleteIntent(PendingIntent intent) {
-        mDeleteIntent = intent;
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder addAction(int iconId, CharSequence title, PendingIntent intent) {
-        if (mActions.size() == MAX_ACTION_BUTTONS) {
-            throw new IllegalStateException(
-                    "Cannot add more than " + MAX_ACTION_BUTTONS + " actions.");
-        }
-        mActions.add(new Action(iconId, limitLength(title), intent));
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder addSettingsAction(
-            int iconId, CharSequence title, PendingIntent intent) {
-        mSettingsAction = new Action(iconId, limitLength(title), intent);
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setDefaults(int defaults) {
-        mDefaults = defaults;
-        return this;
-    }
-
-    @Override
-    public NotificationBuilder setVibrate(long[] pattern) {
-        mVibratePattern = Arrays.copyOf(pattern, pattern.length);
-        return this;
-    }
-
     /**
      * If there are actions, shows the button related views, and adds a button for each action.
      */
     private void addActionButtons(RemoteViews bigView) {
         // Remove the existing buttons in case an existing notification is being updated.
         bigView.removeAllViews(R.id.buttons);
 
         // Always set the visibility of the views associated with the action buttons. The current
         // visibility state is not known as perhaps an existing notification is being updated.
         int visibility = mActions.isEmpty() ? View.GONE : View.VISIBLE;
@@ skipping 62 matching lines @@
 
         // The input bitmap is immutable, so the output drawable will be a different instance from
         // the input drawable if the work profile badge was applied.
         if (inputDrawable != outputDrawable && outputDrawable instanceof BitmapDrawable) {
             view.setImageViewBitmap(
                     R.id.work_profile_badge, ((BitmapDrawable) outputDrawable).getBitmap());
             view.setViewVisibility(R.id.work_profile_badge, View.VISIBLE);
         }
     }
 
-    @Nullable
-    private static CharSequence limitLength(@Nullable CharSequence input) {
-        if (input == null) {
-            return input;
-        }
-        if (input.length() > MAX_CHARSEQUENCE_LENGTH) {
-            return input.subSequence(0, MAX_CHARSEQUENCE_LENGTH);
-        }
-        return input;
-    }
-
     /**
      * Scales down the maximum number of displayed lines in the body text if font scaling is greater
      * than 1.0. Never scales up the number of lines, as on some devices the notification text is
      * rendered in dp units (which do not scale) and additional lines could lead to cropping at the
      * bottom of the notification.
      *
      * @param fontScale The current system font scaling factor.
      * @return The number of lines to be displayed.
      */
     @VisibleForTesting
@@ skipping 32 matching lines @@
                 TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, value, displayMetrics));
     }
 
     /**
      * Whether to use the Material look and feel or fall back to Holo.
      */
     private static boolean useMaterial() {
         return Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP;
     }
 }