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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerDocument.java

Issue 2114513002: Fade theme color in tab switcher (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments & fix coloring invisible LayoutTab Created 4 years, 5 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/layouts/LayoutManagerDocument.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerDocument.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerDocument.java
index f10e88f85414572ee9caeadaf3a9617ffc333856..ac0a0931411ff22bbdcebaa4eb65b795b1c85f66 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerDocument.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerDocument.java
@@ -262,10 +262,11 @@ public class LayoutManagerDocument extends LayoutManager
int themeColor = tab.getThemeColor();
boolean canUseLiveTexture =
tab.getContentViewCore() != null && !tab.isShowingSadTab() && !isNativePage;
- layoutTab.initFromHost(tab.getBackgroundColor(), tab.shouldStall(), canUseLiveTexture,
- themeColor, ColorUtils.getTextBoxColorForToolbarBackground(
+ boolean needsUpdate = layoutTab.initFromHost(tab.getBackgroundColor(), tab.shouldStall(),
+ canUseLiveTexture, themeColor, ColorUtils.getTextBoxColorForToolbarBackground(
mContext.getResources(), tab, themeColor),
ColorUtils.getTextBoxAlphaForToolbarBackground(tab));
+ if (needsUpdate) requestUpdate();
mHost.requestRender();
}

Powered by Google App Engine
This is Rietveld 408576698