A drop-down should take a line to be always aligned with text-fields.

chrome/android/java/src/org/chromium/chrome/browser/payments/ui/EditorView.java

 // Copyright 2016 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.chrome.browser.payments.ui;
 
 import android.animation.Animator;
 import android.animation.AnimatorListenerAdapter;
 import android.animation.AnimatorSet;
 import android.animation.ObjectAnimator;
@@ skipping 303 matching lines @@
         mFieldViews.clear();
         mEditableTextFields.clear();
         mDropdownFields.clear();
 
         // Add Views for each of the {@link EditorFields}.
         for (int i = 0; i < mEditorModel.getFields().size(); i++) {
             EditorFieldModel fieldModel = mEditorModel.getFields().get(i);
             EditorFieldModel nextFieldModel = null;
 
             boolean isLastField = i == mEditorModel.getFields().size() - 1;
-            boolean useFullLine = fieldModel.isFullLine();
-            if (!isLastField && !useFullLine) {
+            // Dropdown fields will take a full line
+            boolean useFullLine = fieldModel.isFullLine() || isLastField
+                    || (fieldModel.getInputTypeHint() == EditorFieldModel.INPUT_TYPE_HINT_DROPDOWN);
+
+            if (!useFullLine) {
                 // If the next field isn't full, stretch it out.
                 nextFieldModel = mEditorModel.getFields().get(i + 1);
-                if (nextFieldModel.isFullLine()) useFullLine = true;
+                useFullLine = nextFieldModel.isFullLine()

[sebsg, 2017/04/03 19:13:36]

There is small change in behavior in your function. In the previous behavior, if
useFullLine was true, it could never be set to false in with the next field.
with your new one, if useFullLine was true, but the nextField is not a full line
nor a dropdown, you will set it to false. 

Maybe that was intended, if that was the case, all good. Otherwise, you should
probably do something like 

if (nextFieldModel.isFullLine() || nextFieldModel.getInputTypeHint() == ...)
  useFullLine = true;

[Parastoo, 2017/04/03 19:41:29]

My mistake :P

On 2017/04/03 19:13:36, sebsg wrote:
Done.

+                        || (nextFieldModel.getInputTypeHint()
+                                   == EditorFieldModel.INPUT_TYPE_HINT_DROPDOWN);
             }
 
-            if (useFullLine || isLastField) {
+            if (useFullLine) {
                 addFieldViewToEditor(mDataView, fieldModel);
             } else {
                 // Create a LinearLayout to put it and the next view side by side.
                 LinearLayout rowLayout = new LinearLayout(mContext);
                 mDataView.addView(rowLayout);
 
                 View firstView = addFieldViewToEditor(rowLayout, fieldModel);
                 View lastView = addFieldViewToEditor(rowLayout, nextFieldModel);
 
                 LinearLayout.LayoutParams firstParams =
@@ skipping 196 matching lines @@
     public List<EditText> getEditableTextFieldsForTest() {
         return mEditableTextFields;
     }
 
     /** @return All dropdown fields in the editor. Used only for tests. */
     @VisibleForTesting
     public List<Spinner> getDropdownFieldsForTest() {
         return mDropdownFields;
     }
 }