Prepare for color input datalist support on Android

Source/web/ChromeClientImpl.h

Index: Source/web/ChromeClientImpl.h
diff --git a/Source/web/ChromeClientImpl.h b/Source/web/ChromeClientImpl.h
index 71c4ace37ea4a143f27397e373e3cff838feb1d2..552fc812244136c5ca88a74896f2b9d6a2ad99bd 100644
--- a/Source/web/ChromeClientImpl.h
+++ b/Source/web/ChromeClientImpl.h
@@ -131,7 +131,7 @@ public:
     virtual void annotatedRegionsChanged();
     virtual bool paintCustomOverhangArea(WebCore::GraphicsContext*, const WebCore::IntRect&, const WebCore::IntRect&, const WebCore::IntRect&);
     virtual PassOwnPtr<WebCore::ColorChooser> createColorChooser(WebCore::ColorChooserClient*, const WebCore::Color&) OVERRIDE;
-    PassOwnPtr<WebColorChooser> createWebColorChooser(WebColorChooserClient*, const WebColor&);
+    PassOwnPtr<WebColorChooser> createWebColorChooser(WebColorChooserClient*, const WebColor&, const Vector<WebCore::Color>&, const Vector<String>&);

[Miguel Garcia, 2013/08/19 14:17:05]

I know blink style does not allow variable names in header files but how do you
tell from looking at this what const Vector<String>& is? I think that sending
one vector with both the colors and the suggestions would be a bit more
readable.

[tkent, 2013/08/20 01:09:45]

In this case, we should add argument names.

Adding non-virtual function to ChromeClientImpl looks strange. 
ColorChooserUIController should have a WebViewClient pointer?

[keishi, 2013/08/23 15:14:22]

Done.

     virtual PassRefPtr<WebCore::DateTimeChooser> openDateTimeChooser(WebCore::DateTimeChooserClient*, const WebCore::DateTimeChooserParameters&) OVERRIDE;
     virtual void runOpenPanel(WebCore::Frame*, PassRefPtr<WebCore::FileChooser>);
     virtual void enumerateChosenDirectory(WebCore::FileChooser*);