Commit 34d33cc6 authored by Marcin Wiacek's avatar Marcin Wiacek Committed by Commit Bot

Cleanup in CertificateViewer.java

Moving two variables into method (cleanup after
https://chromium-review.googlesource.com/c/chromium/src/+/1070202)

Change-Id: I6a622b5fd13e42dd347e03df2536f5af0267dc68
Reviewed-on: https://chromium-review.googlesource.com/1087962Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
Commit-Queue: Marcin Wiącek <marcin@mwiacek.com>
Cr-Commit-Position: refs/heads/master@{#564883}
parent 18927c18
...@@ -49,8 +49,6 @@ class CertificateViewer implements OnItemSelectedListener { ...@@ -49,8 +49,6 @@ class CertificateViewer implements OnItemSelectedListener {
private ArrayList<LinearLayout> mViews; private ArrayList<LinearLayout> mViews;
private CertificateFactory mCertificateFactory; private CertificateFactory mCertificateFactory;
private Dialog mDialog; private Dialog mDialog;
private ScrollView mScrollView;
private Spinner mSpinner;
public CertificateViewer(Context context) { public CertificateViewer(Context context) {
mContext = context; mContext = context;
...@@ -102,14 +100,14 @@ class CertificateViewer implements OnItemSelectedListener { ...@@ -102,14 +100,14 @@ class CertificateViewer implements OnItemSelectedListener {
title.setPadding(mPadding, mPadding, mPadding, mPadding / 2); title.setPadding(mPadding, mPadding, mPadding, mPadding / 2);
dialogContainer.addView(title); dialogContainer.addView(title);
mSpinner = new Spinner(mContext); Spinner spinner = new Spinner(mContext);
ApiCompatibilityUtils.setTextAlignment(mSpinner, View.TEXT_ALIGNMENT_VIEW_START); ApiCompatibilityUtils.setTextAlignment(spinner, View.TEXT_ALIGNMENT_VIEW_START);
mSpinner.setAdapter(arrayAdapter); spinner.setAdapter(arrayAdapter);
mSpinner.setOnItemSelectedListener(this); spinner.setOnItemSelectedListener(this);
mSpinner.setDropDownWidth(ViewGroup.LayoutParams.MATCH_PARENT); spinner.setDropDownWidth(ViewGroup.LayoutParams.MATCH_PARENT);
// Remove padding so that dropdown has same width as the spinner. // Remove padding so that dropdown has same width as the spinner.
mSpinner.setPadding(0, 0, 0, 0); spinner.setPadding(0, 0, 0, 0);
dialogContainer.addView(mSpinner); dialogContainer.addView(spinner);
LinearLayout certContainer = new LinearLayout(mContext); LinearLayout certContainer = new LinearLayout(mContext);
certContainer.setOrientation(LinearLayout.VERTICAL); certContainer.setOrientation(LinearLayout.VERTICAL);
...@@ -120,9 +118,9 @@ class CertificateViewer implements OnItemSelectedListener { ...@@ -120,9 +118,9 @@ class CertificateViewer implements OnItemSelectedListener {
} }
certContainer.addView(certificateView); certContainer.addView(certificateView);
} }
mScrollView = new ScrollView(mContext); ScrollView scrollView = new ScrollView(mContext);
mScrollView.addView(certContainer); scrollView.addView(certContainer);
dialogContainer.addView(mScrollView); dialogContainer.addView(scrollView);
mDialog = new Dialog(mContext); mDialog = new Dialog(mContext);
mDialog.requestWindowFeature(Window.FEATURE_NO_TITLE); mDialog.requestWindowFeature(Window.FEATURE_NO_TITLE);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment