Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(374)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilder.java

Issue 1639163003: Extract NotificationBuilderBase for holding the arguments. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address peter's comments. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilder.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilder.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilder.java
deleted file mode 100644
index 112df41bd988d3508d07f481a68997c12dc7cf1e..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationBuilder.java
+++ /dev/null
@@ -1,92 +0,0 @@
-// 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.graphics.Bitmap;
-
-import javax.annotation.Nullable;
-
-/**
- * Builds a notification using the given inputs.
- */
-public interface NotificationBuilder {
-
- /**
- * Combines all of the options that have been set and returns a new Notification object.
- */
- Notification build();
-
- /**
- * Sets the title text (first row) of the notification.
- */
- NotificationBuilder setTitle(@Nullable CharSequence title);
-
- /**
- * Sets the body text (second row) of the notification.
- */
- NotificationBuilder setBody(@Nullable CharSequence body);
-
- /**
- * Sets the origin text (bottom row) of the notification.
- */
- NotificationBuilder setOrigin(@Nullable CharSequence origin);
-
- /**
- * Sets the text that is displayed in the status bar when the notification first arrives.
- */
- NotificationBuilder setTicker(@Nullable CharSequence tickerText);
-
- /**
- * Sets the large icon that is shown in the notification.
- */
- NotificationBuilder setLargeIcon(@Nullable Bitmap icon);
-
- /**
- * Sets the the small icon that is shown in the notification and in the status bar.
- */
- NotificationBuilder setSmallIcon(int iconId);
-
- /**
- * Sets the PendingIntent to send when the notification is clicked.
- */
- NotificationBuilder setContentIntent(@Nullable PendingIntent intent);
-
- /**
- * Sets the PendingIntent to send when the notification is cleared by the user directly from the
- * notification panel.
- */
- NotificationBuilder setDeleteIntent(@Nullable PendingIntent intent);
-
- /**
- * Adds an action to the notification. Actions are typically displayed as a button adjacent to
- * the notification content.
- */
- NotificationBuilder addAction(
- int iconId, @Nullable CharSequence title, @Nullable PendingIntent intent);
-
- /**
- * Adds an action to the notification for opening the settings screen.
- */
- NotificationBuilder addSettingsAction(int iconId, CharSequence title, PendingIntent intent);
-
- /**
- * Sets the default notification options that will be used.
- * <p>
- * The value should be one or more of the following fields combined with
- * bitwise-or:
- * {@link Notification#DEFAULT_SOUND}, {@link Notification#DEFAULT_VIBRATE},
- * {@link Notification#DEFAULT_LIGHTS}.
- * <p>
- * For all default values, use {@link Notification#DEFAULT_ALL}.
- */
- NotificationBuilder setDefaults(int defaults);
-
- /**
- * Sets the vibration pattern to use.
- */
- NotificationBuilder setVibrate(@Nullable long[] pattern);
-}

Powered by Google App Engine
This is Rietveld 408576698