[Merge M57] Use uniform margin for icon, label and sublabel

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

 // Copyright 2014 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;
 
 import android.content.Context;
 import android.graphics.Color;
 import android.graphics.Typeface;
+import android.support.annotation.Nullable;
 import android.support.v7.content.res.AppCompatResources;
 import android.text.TextUtils;
 import android.util.TypedValue;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
 import android.widget.AbsListView.LayoutParams;
 import android.widget.ArrayAdapter;
 import android.widget.ImageView;
 import android.widget.LinearLayout;
 import android.widget.TextView;
 
 import org.chromium.base.ApiCompatibilityUtils;
 
 import java.util.List;
 import java.util.Set;
 
 /**
  * Dropdown item adapter for DropdownPopupWindow.
  */
 public class DropdownAdapter extends ArrayAdapter<DropdownItem> {
     private final Context mContext;
     private final Set<Integer> mSeparators;
     private final boolean mAreAllItemsEnabled;
     private final Integer mBackgroundColor;
     private final Integer mDividerColor;
     private final Integer mDropdownItemHeight;
+    private final int mLabelVerticalMargin;
+    private final int mLabelHorizontalMargin;
+    private final boolean mHasUniformHorizontalMargin;
 
     /**
      * Creates an {@code ArrayAdapter} with specified parameters.
      * @param context Application context.
      * @param items List of labels and icons to display.
      * @param separators Set of positions that separate {@code items}.
      * @param backgroundColor Popup background color, or {@code null} to use default background
      * color. The default color is {@code Color.TRANSPARENT}.
      * @param dividerColor If {@code null}, use the values in colors.xml for the divider
      * between items. Otherwise, uses {@param dividerColor} for the divider between items. Always
      * uses the values in colors.xml for the dark divider for the separators.
      * @param dropdownItemHeight If {@code null}, uses the {@code dropdown_item_height} in
      * dimens.xml. Otherwise, uses {@param dropdownItemHeight}.
+     * @param margin If {@code null}, uses the {@code dropdown_icon_margin} and
+     * {@code dropdown_item_label_margin} in dropdown_item.xml. Otherwise, uses {@param margin}
+     * for uniform margin for icon, label and between icon and label.
      */
     public DropdownAdapter(Context context, List<? extends DropdownItem> items,
-            Set<Integer> separators, Integer backgroundColor, Integer dividerColor,
-            Integer dropdownItemHeight) {
+            Set<Integer> separators, @Nullable Integer backgroundColor,
+            @Nullable Integer dividerColor, @Nullable Integer dropdownItemHeight,
+            @Nullable Integer margin) {
         super(context, R.layout.dropdown_item);
         mContext = context;
         addAll(items);
         mSeparators = separators;
         mAreAllItemsEnabled = checkAreAllItemsEnabled();
         mBackgroundColor = backgroundColor;
         mDividerColor = dividerColor;
         mDropdownItemHeight = dropdownItemHeight;
+        mLabelVerticalMargin = context.getResources().getDimensionPixelSize(
+                R.dimen.dropdown_item_label_margin);
+        if (margin == null) {
+            mLabelHorizontalMargin = mLabelVerticalMargin;
+            mHasUniformHorizontalMargin = false;
+        } else {
+            // We use a uniform margin between start and the icon, between icon and the label and
+            // between the label and the end.
+            // <-- margin -->|icon|<-- margin -->|label|<-- margin -->
+            // <-- margin -->|label|<-- margin -->
+            mLabelHorizontalMargin = (int) TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP,
+                    margin, context.getResources().getDisplayMetrics());
+            mHasUniformHorizontalMargin = true;
+        }
     }
 
     private boolean checkAreAllItemsEnabled() {
         for (int i = 0; i < getCount(); i++) {
             DropdownItem item = getItem(i);
             if (item.isEnabled() && !item.isGroupHeader()) {
                 return false;
             }
         }
         return true;
@@ skipping 51 matching lines @@
         } else {
             wrapper.setOrientation(LinearLayout.VERTICAL);
         }
 
         wrapper.setLayoutParams(new LinearLayout.LayoutParams(0, height, 1));
 
         TextView labelView = (TextView) layout.findViewById(R.id.dropdown_label);
         labelView.setText(item.getLabel());
         labelView.setSingleLine(!item.isMultilineLabel());
 
+        LinearLayout.LayoutParams layoutParams;
         if (item.isLabelAndSublabelOnSameLine()) {
-            labelView.setLayoutParams(
-                    new LinearLayout.LayoutParams(0, LayoutParams.WRAP_CONTENT, 1));
+            layoutParams = new LinearLayout.LayoutParams(0, LayoutParams.WRAP_CONTENT, 1);
         } else {
-            LinearLayout.LayoutParams layoutParams = new LinearLayout.LayoutParams(
-                    LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT);
-            int labelMargin = mContext.getResources().getDimensionPixelSize(
-                    R.dimen.dropdown_item_label_margin);
-            ApiCompatibilityUtils.setMarginStart(layoutParams, labelMargin);
-            ApiCompatibilityUtils.setMarginEnd(layoutParams, labelMargin);
-            labelView.setLayoutParams(layoutParams);
+            layoutParams = new LinearLayout.LayoutParams(LayoutParams.WRAP_CONTENT,
+                    LayoutParams.WRAP_CONTENT);
+            if (item.getIconId() == DropdownItem.NO_ICON || !mHasUniformHorizontalMargin) {
+                ApiCompatibilityUtils.setMarginStart(layoutParams, mLabelHorizontalMargin);
+            }
+            ApiCompatibilityUtils.setMarginEnd(layoutParams, mLabelHorizontalMargin);
             if (item.isMultilineLabel()) {
-                // If there is a multiline label, we add extra padding top and bottom because
+                // If there is a multiline label, we add extra padding at the top and bottom because
                 // WRAP_CONTENT, defined above for multiline labels, leaves none.
                 int existingStart = ApiCompatibilityUtils.getPaddingStart(labelView);
                 int existingEnd = ApiCompatibilityUtils.getPaddingEnd(labelView);
-                ApiCompatibilityUtils.setPaddingRelative(
-                        labelView, existingStart, labelMargin, existingEnd, labelMargin);
+                ApiCompatibilityUtils.setPaddingRelative(labelView,
+                        existingStart, mLabelVerticalMargin, existingEnd, mLabelVerticalMargin);
             }
         }
 
+        labelView.setLayoutParams(layoutParams);
         labelView.setEnabled(item.isEnabled());
         if (item.isGroupHeader() || item.isBoldLabel()) {
             labelView.setTypeface(null, Typeface.BOLD);
         } else {
             labelView.setTypeface(null, Typeface.NORMAL);
         }
 
         labelView.setTextColor(ApiCompatibilityUtils.getColor(
                 mContext.getResources(), item.getLabelFontColorResId()));
         labelView.setTextSize(TypedValue.COMPLEX_UNIT_PX,
                 mContext.getResources().getDimension(item.getLabelFontSizeResId()));
 
         TextView sublabelView = (TextView) layout.findViewById(R.id.dropdown_sublabel);
         CharSequence sublabel = item.getSublabel();
         if (TextUtils.isEmpty(sublabel)) {
             sublabelView.setVisibility(View.GONE);
         } else {
+            sublabelView.setLayoutParams(layoutParams);
             sublabelView.setText(sublabel);
             sublabelView.setTextSize(TypedValue.COMPLEX_UNIT_PX,
                     mContext.getResources().getDimension(item.getSublabelFontSizeResId()));
             sublabelView.setVisibility(View.VISIBLE);
         }
 
         ImageView iconViewStart = (ImageView) layout.findViewById(R.id.start_dropdown_icon);
         ImageView iconViewEnd = (ImageView) layout.findViewById(R.id.end_dropdown_icon);
         if (item.isIconAtStart()) {
             iconViewEnd.setVisibility(View.GONE);
         } else {
             iconViewStart.setVisibility(View.GONE);
         }
 
         ImageView iconView = item.isIconAtStart() ? iconViewStart : iconViewEnd;
         if (item.getIconId() == DropdownItem.NO_ICON) {
             iconView.setVisibility(View.GONE);
         } else {
             int iconSizeResId = item.getIconSizeResId();
             int iconSize = iconSizeResId == 0
                     ? LayoutParams.WRAP_CONTENT
                     : mContext.getResources().getDimensionPixelSize(iconSizeResId);
-            ViewGroup.MarginLayoutParams layoutParams =
+            ViewGroup.MarginLayoutParams iconLayoutParams =
                     (ViewGroup.MarginLayoutParams) iconView.getLayoutParams();
-            layoutParams.width = iconSize;
-            layoutParams.height = iconSize;
-            int iconMargin =
-                    mContext.getResources().getDimensionPixelSize(item.getIconMarginResId());
-            ApiCompatibilityUtils.setMarginStart(layoutParams, iconMargin);
-            ApiCompatibilityUtils.setMarginEnd(layoutParams, iconMargin);
-            iconView.setLayoutParams(layoutParams);
+            iconLayoutParams.width = iconSize;
+            iconLayoutParams.height = iconSize;
+            int iconMargin = mHasUniformHorizontalMargin ? mLabelHorizontalMargin
+                    : mContext.getResources().getDimensionPixelSize(item.getIconMarginResId());
+            ApiCompatibilityUtils.setMarginStart(iconLayoutParams, iconMargin);
+            ApiCompatibilityUtils.setMarginEnd(iconLayoutParams, iconMargin);
+            iconView.setLayoutParams(iconLayoutParams);
             iconView.setImageDrawable(AppCompatResources.getDrawable(mContext, item.getIconId()));
             iconView.setVisibility(View.VISIBLE);
         }
 
         return layout;
     }
 
     @Override
     public boolean areAllItemsEnabled() {
         return mAreAllItemsEnabled;
     }
 
     @Override
     public boolean isEnabled(int position) {
         if (position < 0 || position >= getCount()) return false;
         DropdownItem item = getItem(position);
         return item.isEnabled() && !item.isGroupHeader();
     }
 }