Show Ash like <select> popup on Android tablets

content/browser/android/content_view_core_impl.cc

Index: content/browser/android/content_view_core_impl.cc
diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc
index 486615af68b2dd084e093516518c9e86e06f061f..6a65ddf2ee167e919346c241e67922b3c1173336 100644
--- a/content/browser/android/content_view_core_impl.cc
+++ b/content/browser/android/content_view_core_impl.cc
@@ -516,12 +516,15 @@ void ContentViewCoreImpl::OnBackgroundColorChanged(SkColor color) {
 }
 
 void ContentViewCoreImpl::ShowSelectPopupMenu(
-    const std::vector<MenuItem>& items, int selected_item, bool multiple) {
+    const gfx::Rect& bounds, const std::vector<MenuItem>& items,

[Ted C, 2014/04/12 01:10:27]

this formatting wasn't correct before, it should actually have a param on each
line since it doesn't all fit on one.

[keishi, 2014/04/15 14:41:07]

Done.

+    int selected_item, double item_font_size, bool multiple) {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> j_obj = java_ref_.get(env);
   if (j_obj.is_null())
     return;
 
+  ScopedJavaLocalRef<jobject> bounds_rect(CreateJavaRect(env, bounds));
+
   // For multi-select list popups we find the list of previous selections by
   // iterating through the items. But for single selection popups we take the
   // given |selected_item| as is.
@@ -559,8 +562,12 @@ void ContentViewCoreImpl::ShowSelectPopupMenu(
   ScopedJavaLocalRef<jobjectArray> items_array(
       base::android::ToJavaArrayOfStrings(env, labels));
   Java_ContentViewCore_showSelectPopup(env, j_obj.obj(),
-                                       items_array.obj(), enabled_array.obj(),
-                                       multiple, selected_array.obj());
+                                       bounds_rect.obj(),
+                                       items_array.obj(),
+                                       enabled_array.obj(),
+                                       item_font_size,
+                                       multiple,
+                                       selected_array.obj());
 }
 
 void ContentViewCoreImpl::HideSelectPopupMenu() {