Implement Android UI of web notification inline replies

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

Index: chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java
index 3c17745178f4f4808019d2c5eba6ccb738a54370..2e0e991913a3757ff992fa18cb5e17639faf4f87 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/notifications/CustomNotificationBuilder.java
@@ -139,10 +139,10 @@ public class CustomNotificationBuilder extends NotificationBuilderBase {
         builder.setLargeIcon(mLargeIcon);
         setSmallIconOnBuilder(builder, mSmallIconId, mSmallIconBitmap);
         for (Action action : mActions) {
-            addActionToBuilder(builder, action);
+            addActionToBuilder(builder, action, mDefaultPlaceholder);
         }
         if (mSettingsAction != null) {
-            addActionToBuilder(builder, mSettingsAction);
+            addActionToBuilder(builder, mSettingsAction, null /* defaultPlaceholder */);

[Michael van Ouwerkerk, 2016/04/07 14:19:20]

My point is that these basically identical calls are somehow different between
CustomNotificationBuilder and StandardNotificationBuilder. That's confusing to
read, even if what is passed here is irrelevant.

         }
 
         Notification notification = builder.build();