summaryrefslogtreecommitdiff
path: root/core/java/android/widget/AdapterView.java
diff options
context:
space:
mode:
authorAlan Viverette <alanv@google.com>2015-06-01 16:41:42 -0700
committerAlan Viverette <alanv@google.com>2015-06-01 16:41:42 -0700
commit376c32f268be25633da2305099c82451a539d6bf (patch)
tree36b1514c2c006a9d263ece58ea729e19ad3fc7d8 /core/java/android/widget/AdapterView.java
parentbdd500ecd4ceea7d4e4a06ece08c9b9fd5d8aec5 (diff)
Always send VIEW_CLICKED event from AdapterView.performItemClick()
This makes it consistent with View.performClick(). Bug: 19574976 Change-Id: I00b222d56aa6fac03c2e6ead0604ee2ff3a6969d
Diffstat (limited to 'core/java/android/widget/AdapterView.java')
-rw-r--r--core/java/android/widget/AdapterView.java13
1 files changed, 8 insertions, 5 deletions
diff --git a/core/java/android/widget/AdapterView.java b/core/java/android/widget/AdapterView.java
index 54e3996927ed..696271105639 100644
--- a/core/java/android/widget/AdapterView.java
+++ b/core/java/android/widget/AdapterView.java
@@ -304,16 +304,19 @@ public abstract class AdapterView<T extends Adapter> extends ViewGroup {
* called, false otherwise is returned.
*/
public boolean performItemClick(View view, int position, long id) {
+ final boolean result;
if (mOnItemClickListener != null) {
playSoundEffect(SoundEffectConstants.CLICK);
mOnItemClickListener.onItemClick(this, view, position, id);
- if (view != null) {
- view.sendAccessibilityEvent(AccessibilityEvent.TYPE_VIEW_CLICKED);
- }
- return true;
+ result = true;
+ } else {
+ result = false;
}
- return false;
+ if (view != null) {
+ view.sendAccessibilityEvent(AccessibilityEvent.TYPE_VIEW_CLICKED);
+ }
+ return result;
}
/**