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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/TabListSceneLayer.java

Issue 2862113003: [Home] Disable theme colors in toolbar (Closed)
Patch Set: jk Created 3 years, 7 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/compositor/scene_layer/TabListSceneLayer.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/TabListSceneLayer.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/TabListSceneLayer.java
index 8d5196159b88e77f6bbd87b4f3553a3b36b06578..10ae9a7fa52f3e7bbd927fb15602d35d7bb5a08d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/TabListSceneLayer.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/TabListSceneLayer.java
@@ -68,20 +68,28 @@ public class TabListSceneLayer extends SceneLayer {
? ApiCompatibilityUtils.getColor(res, R.color.incognito_primary_color)
: ApiCompatibilityUtils.getColor(res, R.color.default_primary_color);
- int closeButtonColor = ColorUtils.getThemedAssetColor(t.getToolbarBackgroundColor(),
- t.isIncognito());
+ int toolbarBackgroundColor = t.getToolbarBackgroundColor();
+ int textBoxBackgroundColor = t.getTextBoxBackgroundColor();
+ float textBoxAlpha = t.getTextBoxAlpha();
+ if (t.getForceDefaultThemeColor()) {
+ toolbarBackgroundColor = defaultThemeColor;
+ textBoxBackgroundColor = ColorUtils.getTextBoxColorForToolbarBackground(
+ res, false, toolbarBackgroundColor);
+ textBoxAlpha = t.isIncognito() ? textBoxAlpha : 1f;
+ }
+
+ int closeButtonColor =
+ ColorUtils.getThemedAssetColor(toolbarBackgroundColor, t.isIncognito());
int borderColorResource =
t.isIncognito() ? R.color.tab_back_incognito : R.color.tab_back;
// TODO(dtrainor, clholgat): remove "* dpToPx" once the native part fully supports dp.
nativePutTabLayer(mNativePtr, t.getId(), R.id.control_container,
- R.drawable.btn_tab_close,
- R.drawable.tabswitcher_border_frame_shadow,
+ R.drawable.btn_tab_close, R.drawable.tabswitcher_border_frame_shadow,
R.drawable.tabswitcher_border_frame_decoration, R.drawable.logo_card_back,
R.drawable.tabswitcher_border_frame,
- R.drawable.tabswitcher_border_frame_inner_shadow,
- t.canUseLiveTexture(), fullscreenManager.areBrowserControlsAtBottom(),
- t.getBackgroundColor(),
+ R.drawable.tabswitcher_border_frame_inner_shadow, t.canUseLiveTexture(),
+ fullscreenManager.areBrowserControlsAtBottom(), t.getBackgroundColor(),
ApiCompatibilityUtils.getColor(res, borderColorResource), t.isIncognito(),
layout.getOrientation() == Orientation.PORTRAIT, t.getRenderX() * dpToPx,
t.getRenderY() * dpToPx, t.getScaledContentWidth() * dpToPx,
@@ -96,9 +104,9 @@ public class TabListSceneLayer extends SceneLayer {
t.getShadowOpacity() * decoration, t.getBorderCloseButtonAlpha() * decoration,
LayoutTab.CLOSE_BUTTON_WIDTH_DP * dpToPx, t.getStaticToViewBlend(),
t.getBorderScale(), t.getSaturation(), t.getBrightness(), t.showToolbar(),
- defaultThemeColor, t.getToolbarBackgroundColor(), closeButtonColor,
+ defaultThemeColor, toolbarBackgroundColor, closeButtonColor,
t.anonymizeToolbar(), t.isTitleNeeded(), R.drawable.card_single,
- t.getTextBoxBackgroundColor(), t.getTextBoxAlpha(), t.getToolbarAlpha(),
+ textBoxBackgroundColor, textBoxAlpha, t.getToolbarAlpha(),
t.getToolbarYOffset() * dpToPx, t.getSideBorderScale(),
t.insetBorderVertical());
}

Powered by Google App Engine
This is Rietveld 408576698