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

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

Issue 2158853004: Do not include toolbar shadow for UIResourceLayer snapshot (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update comment 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/components/LayoutTab.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/components/LayoutTab.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/components/LayoutTab.java
index f541c80a93901d1154071525d637769b8df5a2dd..4acece77479a2f5d06c3ab02301d33dbb300ee68 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/components/LayoutTab.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/components/LayoutTab.java
@@ -68,6 +68,7 @@ public class LayoutTab implements ChromeAnimation.Animatable<LayoutTab.Property>
private final int mId;
private final boolean mIsIncognito;
+ private boolean mNeedsToolbarShadow;
// Fields initialized in init()
private float mScale;
@@ -212,6 +213,7 @@ public class LayoutTab implements ChromeAnimation.Animatable<LayoutTab.Property>
* Initializes the {@link LayoutTab} from data extracted from a {@link Tab}.
* As this function may be expensive and can be delayed we initialize it as a separately.
*
+ * @param needsToolbarShadow Whether the tab will need the toolbar shadow.
* @param backgroundColor The color of the page background.
* @param fallbackThumbnailId The id of a cached thumbnail to show if the current
* thumbnail is unavailable, or {@link Tab.INVALID_TAB_ID}
@@ -221,8 +223,10 @@ public class LayoutTab implements ChromeAnimation.Animatable<LayoutTab.Property>
* @param canUseLiveTexture Whether the tab can use a live texture when being displayed.
* @return True if the init requires the compositor to update.
*/
- public boolean initFromHost(int backgroundColor, boolean shouldStall, boolean canUseLiveTexture,
- int toolbarBackgroundColor, int textBoxBackgroundColor, float textBoxAlpha) {
+ public boolean initFromHost(boolean needsToolbarShadow, int backgroundColor,
+ boolean shouldStall, boolean canUseLiveTexture, int toolbarBackgroundColor,
+ int textBoxBackgroundColor, float textBoxAlpha) {
+ mNeedsToolbarShadow = needsToolbarShadow;
mBackgroundColor = backgroundColor;
boolean needsUpdate = false;
@@ -496,6 +500,13 @@ public class LayoutTab implements ChromeAnimation.Animatable<LayoutTab.Property>
}
/**
+ * @return Whether the underlying tab needs to draw the toolbar shadow.
+ */
+ public boolean needsToolbarShadow() {
+ return mNeedsToolbarShadow;
+ }
+
+ /**
* @param y The vertical draw position.
*/
public void setY(float y) {

Powered by Google App Engine
This is Rietveld 408576698