Index: chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
index c4bff45fd596004c1c2fb7a208ec5a238c7c2a5b..a6d56e28afb94ba6a86fe87bef73ab742e00054f 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
@@ -131,7 +131,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
} |
@Override |
- public void addCustomActionButton(Drawable drawable, String description, |
+ public void setCustomActionButton(Drawable drawable, String description, |
OnClickListener listener) { |
Resources resources = getResources(); |
@@ -152,6 +152,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
mCustomActionButton.setContentDescription(description); |
mCustomActionButton.setOnClickListener(listener); |
mCustomActionButton.setVisibility(VISIBLE); |
+ updateButtonsTint(); |
} |
/** |
@@ -265,15 +266,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
public void updateVisualsForState() { |
Resources resources = getResources(); |
updateSecurityIcon(getSecurityLevel()); |
- ColorStateList colorStateList = resources.getColorStateList(mUseDarkColors |
- ? R.color.dark_mode_tint : R.color.light_mode_tint); |
- mMenuButton.setTint(colorStateList); |
- if (mCloseButton.getDrawable() instanceof TintedDrawable) { |
- ((TintedDrawable) mCloseButton.getDrawable()).setTint(colorStateList); |
- } |
- if (mCustomActionButton.getDrawable() instanceof TintedDrawable) { |
- ((TintedDrawable) mCustomActionButton.getDrawable()).setTint(colorStateList); |
- } |
+ updateButtonsTint(); |
mUrlBar.setUseDarkTextColors(mUseDarkColors); |
int titleTextColor = mUseDarkColors ? resources.getColor(R.color.url_emphasis_default_text) |
@@ -288,6 +281,19 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
} |
} |
+ private void updateButtonsTint() { |
+ Resources resources = getResources(); |
+ ColorStateList colorStateList = resources.getColorStateList(mUseDarkColors |
+ ? R.color.dark_mode_tint : R.color.light_mode_tint); |
+ mMenuButton.setTint(colorStateList); |
+ if (mCloseButton.getDrawable() instanceof TintedDrawable) { |
+ ((TintedDrawable) mCloseButton.getDrawable()).setTint(colorStateList); |
+ } |
+ if (mCustomActionButton.getDrawable() instanceof TintedDrawable) { |
+ ((TintedDrawable) mCustomActionButton.getDrawable()).setTint(colorStateList); |
+ } |
+ } |
+ |
@Override |
public void setMenuButtonHelper(final AppMenuButtonHelper helper) { |
mMenuButton.setOnTouchListener(new OnTouchListener() { |