Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(289)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarTablet.java

Issue 2779543005: Add support for highlighting menu items (Closed)
Patch Set: Moved PulseDrawable Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarTablet.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarTablet.java b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarTablet.java
index ff297ad114b55abaacdcd3cdc2294b21b8562cd4..9a525cba863a1bad8979d80da35cbb669b14db99 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarTablet.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarTablet.java
@@ -356,6 +356,7 @@ public class ToolbarTablet extends ToolbarLayout implements OnClickListener {
mUseLightColorAssets = incognito;
}
+ setMenuButtonHighlightDrawable(mHighlightingMenu);
updateNtp();
}
@@ -463,6 +464,7 @@ public class ToolbarTablet extends ToolbarLayout implements OnClickListener {
setAppMenuUpdateBadgeToVisible(false);
}
}
+ setMenuButtonHighlightDrawable(mHighlightingMenu);
}
@Override
@@ -502,6 +504,11 @@ public class ToolbarTablet extends ToolbarLayout implements OnClickListener {
}
@Override
+ public boolean useLightDrawables() {
+ return mUseLightColorAssets;
+ }
+
+ @Override
public void showAppMenuUpdateBadge() {
super.showAppMenuUpdateBadge();
if (!mIsInTabSwitcherMode) {

Powered by Google App Engine
This is Rietveld 408576698