Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3357)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/autofill/AutofillDialogTitleView.java

Issue 14801009: [rAc Android dialog] Logo, banner, text, alignment fixes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/autofill/AutofillDialogTitleView.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/autofill/AutofillDialogTitleView.java b/chrome/android/java/src/org/chromium/chrome/browser/autofill/AutofillDialogTitleView.java
index c17b2b450860f8b43c61aa76cfaf6dee53094d38..e17a7ba5d02f92831550a5ea317829faf0eea442 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/autofill/AutofillDialogTitleView.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/autofill/AutofillDialogTitleView.java
@@ -8,6 +8,7 @@ import java.util.List;
import android.content.Context;
import android.view.LayoutInflater;
+import android.view.View;
import android.widget.ArrayAdapter;
import android.widget.FrameLayout;
import android.widget.Spinner;
@@ -52,16 +53,32 @@ public class AutofillDialogTitleView extends FrameLayout {
Spinner accountsSpinner = (Spinner) findViewById(R.id.accounts_spinner);
if (selectedAccountIndex >= 0) {
accountsSpinner.setSelection(selectedAccountIndex);
+ accountsSpinner.setVisibility(View.VISIBLE);
+ View logo = findViewById(R.id.accounts_logo);
+ boolean isWallet = accountNames.size() > 1 && selectedAccountIndex == 0;
+ logo.setVisibility(isWallet ? View.VISIBLE : View.GONE);
+ } else {
+ hideLogoAndAccountChooserVisibility();
}
}
/**
+ * Hide the contents of the title view.
+ */
+ public void hideLogoAndAccountChooserVisibility() {
+ findViewById(R.id.accounts_spinner).setVisibility(View.GONE);
+ findViewById(R.id.accounts_logo).setVisibility(View.GONE);
+ }
+
+ /**
* Enables/disables the account chooser.
* @param enabled True if the account chooser is enabled.
*/
public void setAccountChooserEnabled(boolean enabled) {
Spinner accountsSpinner = (Spinner) findViewById(R.id.accounts_spinner);
accountsSpinner.setEnabled(enabled);
+ View logo = findViewById(R.id.accounts_logo);
+ logo.setEnabled(enabled);
}
/**

Powered by Google App Engine
This is Rietveld 408576698