Add support for color input datalist on Android

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

Index: ui/android/java/src/org/chromium/ui/ColorSuggestion.java
diff --git a/ui/android/java/src/org/chromium/ui/ColorSuggestion.java b/ui/android/java/src/org/chromium/ui/ColorSuggestion.java
new file mode 100644
index 0000000000000000000000000000000000000000..025597c9840ac22235b51ff3b6d035cf2e8560fd
--- /dev/null
+++ b/ui/android/java/src/org/chromium/ui/ColorSuggestion.java
@@ -0,0 +1,26 @@
+// Copyright 2013 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.ui;
+
+/**
+ * Color suggestion container used to store information for each color button.

[newt (away), 2013/08/26 20:23:20]

I'd mention this is for the color picker UI

[keishi, 2013/08/29 03:40:47]

Done.

+ */
+public class ColorSuggestion {
+    final int mValueAsColor;
+    final String mValueAsText;
+    final String mLabel;
+
+    /**
+     * Constructs a color suggestion container.
+     * @param valueAsColor The suggested color.
+     * @param valueAsText The suggested color value as a string.

[palmer, 2013/08/26 20:38:01]

Is this the *name* of the color, or a CSS value like "#FFFFFFFF"? We should not
send such strings over IPC because inevitably they will get interpolated into
CSS.

Also, it's just redundant to pass the color both as an int and as a CSS value
string. Just use the int.

+     * @param label The label for the suggestion.
+     */
+    public ColorSuggestion(int valueAsColor, String valueAsText, String label) {

[newt (away), 2013/08/26 20:23:20]

what's the difference between "valueAsText" and "label"? Could you make it clear
in the javadoc?

Perhaps the member variables could be named something like: mColor, mLabel, mId?

+        mValueAsColor = valueAsColor;
+        mValueAsText = valueAsText;
+        mLabel = label;
+    }
+}