summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Adam Powell <adamp@google.com> 2013-07-12 21:36:07 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2013-07-12 21:36:07 +0000
commitcc5b49f07ff856677cdfdbb0ecff1dcaeaaedc66 (patch)
treed6f00dacc7432613730c846e6c21280ac7d294dc
parent414e204199396f5754caaf3f3a786a21904ec472 (diff)
parentb4c8ba49ac6ccce8e5c7904741f4adff05e37640 (diff)
Merge "Fix MenuItemImpl#hasCollapsibleActionView for action providers"
-rw-r--r--core/java/com/android/internal/view/menu/MenuItemImpl.java10
1 files changed, 8 insertions, 2 deletions
diff --git a/core/java/com/android/internal/view/menu/MenuItemImpl.java b/core/java/com/android/internal/view/menu/MenuItemImpl.java
index 39078ca2746d..3d6b1166d9ce 100644
--- a/core/java/com/android/internal/view/menu/MenuItemImpl.java
+++ b/core/java/com/android/internal/view/menu/MenuItemImpl.java
@@ -616,7 +616,7 @@ public final class MenuItemImpl implements MenuItem {
@Override
public boolean expandActionView() {
- if ((mShowAsAction & SHOW_AS_ACTION_COLLAPSE_ACTION_VIEW) == 0 || mActionView == null) {
+ if (hasCollapsibleActionView()) {
return false;
}
@@ -653,7 +653,13 @@ public final class MenuItemImpl implements MenuItem {
}
public boolean hasCollapsibleActionView() {
- return (mShowAsAction & SHOW_AS_ACTION_COLLAPSE_ACTION_VIEW) != 0 && mActionView != null;
+ if ((mShowAsAction & SHOW_AS_ACTION_COLLAPSE_ACTION_VIEW) != 0) {
+ if (mActionView == null && mActionProvider != null) {
+ mActionView = mActionProvider.onCreateActionView(this);
+ }
+ return mActionView != null;
+ }
+ return false;
}
public void setActionViewExpanded(boolean isExpanded) {