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

Unified Diff: chrome/browser/android/compositor/scene_layer/tab_list_scene_layer.cc

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/browser/android/compositor/scene_layer/tab_list_scene_layer.cc
diff --git a/chrome/browser/android/compositor/scene_layer/tab_list_scene_layer.cc b/chrome/browser/android/compositor/scene_layer/tab_list_scene_layer.cc
index a942971bd876fa20a420a2f3c64eda3d7953aa22..a4f5b811dcfab649fb8694b1f86624ff0e00d276 100644
--- a/chrome/browser/android/compositor/scene_layer/tab_list_scene_layer.cc
+++ b/chrome/browser/android/compositor/scene_layer/tab_list_scene_layer.cc
@@ -76,6 +76,7 @@ void TabListSceneLayer::PutTabLayer(
const JavaParamRef<jobject>& jobj,
jint id,
jint toolbar_resource_id,
+ jint toolbar_shadow_resource_id,
jint close_button_resource_id,
jint shadow_resource_id,
jint contour_resource_id,
@@ -124,13 +125,15 @@ void TabListSceneLayer::PutTabLayer(
jfloat toolbar_y_offset,
jfloat side_border_scale,
jboolean attach_content,
- jboolean inset_border) {
+ jboolean inset_border,
+ jboolean needs_toolbar_shadow) {
scoped_refptr<TabLayer> layer = GetNextLayer(incognito);
// https://crbug.com/517314: GetNextLayer() returns null in some corner cases.
DCHECK(layer);
if (layer) {
layer->SetProperties(
- id, can_use_live_layer, toolbar_resource_id, close_button_resource_id,
+ id, can_use_live_layer, toolbar_resource_id, toolbar_shadow_resource_id,
+ close_button_resource_id,
shadow_resource_id, contour_resource_id, back_logo_resource_id,
border_resource_id, border_inner_shadow_resource_id,
tab_background_color, back_logo_color, is_portrait, x, y, width, height,
@@ -142,7 +145,7 @@ void TabListSceneLayer::PutTabLayer(
default_theme_color, toolbar_background_color, anonymize_toolbar,
toolbar_textbox_resource_id, toolbar_textbox_background_color,
toolbar_textbox_alpha, toolbar_alpha, toolbar_y_offset,
- side_border_scale, attach_content, inset_border);
+ side_border_scale, attach_content, inset_border, needs_toolbar_shadow);
}
if (attach_content) {

Powered by Google App Engine
This is Rietveld 408576698