Http Bad: Add a PopupItemId to identify http warning message

ui/android/java/src/org/chromium/ui/DropdownItem.java

Index: ui/android/java/src/org/chromium/ui/DropdownItem.java
diff --git a/ui/android/java/src/org/chromium/ui/DropdownItem.java b/ui/android/java/src/org/chromium/ui/DropdownItem.java
index dc4468c3fa04ba966352e8720e1ffefd02e72d29..ee9c6abdc2a04e13f2b10f90c92ba343062948cd 100644
--- a/ui/android/java/src/org/chromium/ui/DropdownItem.java
+++ b/ui/android/java/src/org/chromium/ui/DropdownItem.java
@@ -35,4 +35,12 @@ public interface DropdownItem {
      * Returns whether the label should be displayed over multiple lines.
      */
     boolean isMultilineLabel();
+    /**
+     * Returns label's font color.
+     */
+    int getLabelFontColor();

[Ted C, 2016/11/17 17:52:26]

For this and getLabelFontSize, I would add the suffix ResId (resource ID for
short).  Both colors and font sizes can be presented as integers, so this could
cause user confusion if they were to return Color.RED and it wouldn't work as
there is no resource ID associated with it.

I would also update the comments to make it even more abundantly clear that this
is a android resource ID.

[lshang, 2016/11/18 00:03:19]

Good suggestion!

Done.

+    /**
+     * Returns label's font size.
+     */
+    int getLabelFontSize();
 }