Index: chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
index f49b9d20ebc3a6f984c761d7007b66d941af54f0..dcb5268b9f8b1f70e0813f994f5bb205b15e8598 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
@@ -1454,6 +1454,7 @@ public class ToolbarPhone extends ToolbarLayout |
} |
} |
+ // TODO(dtrainor): This is always true when in the tab switcher (crbug.com/710750). |
private boolean isTabSwitcherAnimationRunning() { |
return mTabSwitcherState == ENTERING_TAB_SWITCHER |
|| mTabSwitcherState == EXITING_TAB_SWITCHER; |
@@ -2137,6 +2138,7 @@ public class ToolbarPhone extends ToolbarLayout |
mMenuButton.setTint(mUseLightToolbarDrawables ? mLightModeTint : mDarkModeTint); |
+ setMenuButtonHighlightDrawable(mHighlightingMenu); |
if (mShowMenuBadge && inOrEnteringStaticTab) { |
setAppMenuUpdateBadgeDrawable(mUseLightToolbarDrawables); |
} |
@@ -2180,6 +2182,17 @@ public class ToolbarPhone extends ToolbarLayout |
} |
@Override |
+ public boolean useLightDrawables() { |
+ return mUseLightToolbarDrawables; |
+ } |
+ |
+ @Override |
+ protected void setMenuButtonHighlightDrawable(boolean highlighting) { |
+ highlighting &= !isTabSwitcherAnimationRunning(); |
+ super.setMenuButtonHighlightDrawable(highlighting); |
+ } |
+ |
+ @Override |
public void showAppMenuUpdateBadge() { |
super.showAppMenuUpdateBadge(); |