[Payments] Update autofill/payments spacings

chrome/android/java/res/layout/autofill_profile_editor.xml

Index: chrome/android/java/res/layout/autofill_profile_editor.xml
diff --git a/chrome/android/java/res/layout/autofill_profile_editor.xml b/chrome/android/java/res/layout/autofill_profile_editor.xml
index 559c06081d6b523c7853ee3b8502652ecf5f951e..bbb805326b3965fa8f6d8afb0e2734f55537804a 100644
--- a/chrome/android/java/res/layout/autofill_profile_editor.xml
+++ b/chrome/android/java/res/layout/autofill_profile_editor.xml
@@ -3,46 +3,29 @@
      Use of this source code is governed by a BSD-style license that can be
      found in the LICENSE file. -->
 
-<!-- EditText views in FloatLabelLayout require both hint and contentDescription to be set
-     to work correctly for accessibility because FloatLabelLayout sometimes clear hint. -->
-<!--suppress ContentDescription -->
 <merge
     xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto" >
 
-    <!-- Editable fields for the profile -->
-    <LinearLayout
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:orientation="vertical"
-        android:layout_marginStart="@dimen/pref_autofill_field_horizontal_padding"
-        android:layout_marginEnd="@dimen/pref_autofill_field_horizontal_padding"
-        android:paddingBottom="5dp" >
-        <TextView
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:focusable="true"
-            android:textAppearance="@style/PreferenceFloatLabelTextAppearance"
-            android:text="@string/autofill_profile_editor_country"
-            android:paddingBottom="8dp" />
-        <Spinner
-            android:id="@+id/countries"
-            android:layout_width="fill_parent"
-            android:layout_height="wrap_content" />
-        <View style="@style/PreferenceSpinnerUnderlineView" />
-    </LinearLayout>
+    <!-- Country -->
+    <include layout="@layout/payment_request_editor_dropdown" />
 
+    <!-- Country specific address information
+         Bottom margins are handled by the views that are added to this layout.
+    -->
     <LinearLayout
         android:id="@+id/autofill_profile_widget_root"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:orientation="vertical" >
-    </LinearLayout>
+        android:orientation="vertical" />
 
+    <!-- Phone number -->
     <org.chromium.chrome.browser.widget.CompatibilityTextInputLayout
         android:id="@+id/phone_number_label"
         android:layout_width="match_parent"
-        android:layout_height="wrap_content" >
+        android:layout_height="wrap_content"
+        android:layout_marginBottom="@dimen/pref_autofill_content_spacing" >
+
         <EditText
             android:id="@+id/phone_number_edit"
             android:layout_width="match_parent"
@@ -50,13 +33,18 @@
             android:imeOptions="flagNoExtractUi"
             android:inputType="phone"
             android:singleLine="true"
-            android:hint="@string/autofill_profile_editor_phone_number" />
+            android:hint="@string/autofill_profile_editor_phone_number"
+            android:focusable="true"

[Ian Wen, 2016/07/06 22:04:39]

Remove these since EditTexts are already focusable?

[gone, 2016/07/06 22:08:01]

Done.

+            android:focusableInTouchMode="true" />
+
     </org.chromium.chrome.browser.widget.CompatibilityTextInputLayout>
 
+    <!-- Email address -->
     <org.chromium.chrome.browser.widget.CompatibilityTextInputLayout
         android:id="@+id/email_address_label"
         android:layout_width="match_parent"
         android:layout_height="wrap_content" >
+
         <EditText
             android:id="@+id/email_address_edit"
             android:layout_width="match_parent"
@@ -64,7 +52,10 @@
             android:imeOptions="flagNoExtractUi"
             android:inputType="textEmailAddress"
             android:singleLine="true"
-            android:hint="@string/autofill_profile_editor_email_address" />
+            android:hint="@string/autofill_profile_editor_email_address"
+            android:focusable="true"
+            android:focusableInTouchMode="true" />
+
     </org.chromium.chrome.browser.widget.CompatibilityTextInputLayout>
 
 </merge>