diff options
| author | Jiaquan He <hejq@google.com> | 2017-05-01 14:18:39 -0700 |
|---|---|---|
| committer | Jiaquan He <hejq@google.com> | 2017-05-26 14:31:29 -0700 |
| commit | 1dd48d0b8f2c35f7258c0ea69fe23c9b1435736b (patch) | |
| tree | e7105776d2c4cca46e5084d7522150fc13c186dd /core/java/android/widget/ImageView.java | |
| parent | 0bdf1c9c76c0832d3adbe85bdc0cde4c1e3102ce (diff) | |
Check default focus highlight for ImageView.
This commit specially checks isDefaultFocusHighlightNeeded for
ImageView. We should also check with the content drawable of the
ImageView besides its foreground or background.
Bug: 62141891
Test: cts-tradefed run singleCommand cts --skip-device-info
--skip-preconditions --abi armeabi-v7a -m CtsViewTestCases -t
android.view.cts.View_DefaultFocusHighlightTest#testIsDefaultFocusHighlightNeeded
Change-Id: Iaf12a5863d7760d9361d0196a46de07a9ccda74e
Diffstat (limited to 'core/java/android/widget/ImageView.java')
| -rw-r--r-- | core/java/android/widget/ImageView.java | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/core/java/android/widget/ImageView.java b/core/java/android/widget/ImageView.java index 57818e2cfb55..1dc5b44bed4f 100644 --- a/core/java/android/widget/ImageView.java +++ b/core/java/android/widget/ImageView.java @@ -19,6 +19,7 @@ package android.widget; import android.annotation.DrawableRes; import android.annotation.NonNull; import android.annotation.Nullable; +import android.annotation.TestApi; import android.content.ContentResolver; import android.content.Context; import android.content.res.ColorStateList; @@ -1640,4 +1641,13 @@ public class ImageView extends View { super.encodeProperties(stream); stream.addProperty("layout:baseline", getBaseline()); } + + /** @hide */ + @Override + @TestApi + public boolean isDefaultFocusHighlightNeeded(Drawable background, Drawable foreground) { + final boolean lackFocusState = mDrawable == null || !mDrawable.isStateful() + || !mDrawable.hasFocusStateSpecified(); + return super.isDefaultFocusHighlightNeeded(background, foreground) && lackFocusState; + } } |
