diff options
author | twellington <twellington@chromium.org> | 2015-01-06 19:44:56 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-01-07 03:45:33 +0000 |
commit | 2bdcbe888b0ba79ee55f9d6353ad02d6b67ec256 (patch) | |
tree | f14b016cc23573b958b909be7d98146bcd2e2a9a | |
parent | 4dbfc8480b9a7e8728603a916cbbd8596707c39d (diff) | |
download | chromium_src-2bdcbe888b0ba79ee55f9d6353ad02d6b67ec256.zip chromium_src-2bdcbe888b0ba79ee55f9d6353ad02d6b67ec256.tar.gz chromium_src-2bdcbe888b0ba79ee55f9d6353ad02d6b67ec256.tar.bz2 |
Add contentDescriptions to PreferenceCategoryWithButton icons
BUG=446372
Review URL: https://codereview.chromium.org/807683003
Cr-Commit-Position: refs/heads/master@{#310229}
-rw-r--r-- | chrome/android/java/res/xml/autofill_preferences.xml | 2 | ||||
-rw-r--r-- | chrome/android/java/src/org/chromium/chrome/browser/preferences/PreferenceCategoryWithButton.java | 16 |
2 files changed, 17 insertions, 1 deletions
diff --git a/chrome/android/java/res/xml/autofill_preferences.xml b/chrome/android/java/res/xml/autofill_preferences.xml index c2e0eb7..ee73d31 100644 --- a/chrome/android/java/res/xml/autofill_preferences.xml +++ b/chrome/android/java/res/xml/autofill_preferences.xml @@ -15,6 +15,7 @@ android:enabled="true" android:title="@string/autofill_profiles_title" android:fragment="org.chromium.chrome.browser.preferences.autofill.AutofillProfileEditor" + android:contentDescription="@string/autofill_create_profile" /> <org.chromium.chrome.browser.preferences.PreferenceCategoryWithButton @@ -22,6 +23,7 @@ android:enabled="true" android:title="@string/autofill_credit_cards_title" android:fragment="org.chromium.chrome.browser.preferences.autofill.AutofillCreditCardEditor" + android:contentDescription="@string/autofill_create_credit_card" /> </PreferenceScreen> diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/PreferenceCategoryWithButton.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/PreferenceCategoryWithButton.java index 61521a0..adab1a8 100644 --- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/PreferenceCategoryWithButton.java +++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/PreferenceCategoryWithButton.java @@ -5,8 +5,10 @@ package org.chromium.chrome.browser.preferences; import android.content.Context; +import android.content.res.TypedArray; import android.preference.Preference; import android.preference.PreferenceGroup; +import android.text.TextUtils; import android.util.AttributeSet; import android.view.View; import android.view.View.OnClickListener; @@ -20,10 +22,17 @@ import org.chromium.chrome.R; */ public class PreferenceCategoryWithButton extends PreferenceGroup implements OnClickListener { + private String mContentDescription; + public PreferenceCategoryWithButton(Context context, AttributeSet attrs) { super(context, attrs); setLayoutResource(R.layout.preference_category); setSelectable(false); + + TypedArray a = context.obtainStyledAttributes(attrs, + new int[] {android.R.attr.contentDescription}); + mContentDescription = a.getString(0); + a.recycle(); } @Override @@ -33,7 +42,12 @@ public class PreferenceCategoryWithButton extends PreferenceGroup implements OnC // which needs a top padding of 16dp; we don't want this top padding for // PreferenceCategoryWithButton views. view.setPadding(view.getPaddingLeft(), 0, view.getPaddingRight(), view.getPaddingBottom()); - view.findViewById(android.R.id.icon).setOnClickListener(this); + View button = view.findViewById(android.R.id.icon); + button.setOnClickListener(this); + + if (!TextUtils.isEmpty(mContentDescription)) { + button.setContentDescription(mContentDescription); + } } @Override |