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

Unified Diff: chrome/browser/android/compositor/compositor_view.cc

Issue 1268483004: Plumb the anonymize rect color to toolbar_layer from Tab#getThemeColor (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@tabHasThemeColor
Patch Set: Moved all flow control to Tab Created 5 years, 4 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/compositor_view.cc
diff --git a/chrome/browser/android/compositor/compositor_view.cc b/chrome/browser/android/compositor/compositor_view.cc
index b498aa6e05420e753ac6fea2802da4e85956e1c5..c89995eaaadb34b97dbbff05faa2dfb4a875177d 100644
--- a/chrome/browser/android/compositor/compositor_view.cc
+++ b/chrome/browser/android/compositor/compositor_view.cc
@@ -260,7 +260,8 @@ void CompositorView::UpdateToolbarLayer(JNIEnv* env,
toolbar_layer_->layer()->SetHideLayerAndSubtree(!visible);
if (visible) {
toolbar_layer_->layer()->SetPosition(gfx::PointF(0, top_offset));
- toolbar_layer_->PushResource(resource, false, false, false, brightness);
+ toolbar_layer_->PushResource(
+ resource, SK_ColorWHITE, false, false, brightness);
David Trainor- moved to gerrit 2015/08/26 00:16:40 This is the default toolbar texture shown in the s
Yusuf 2015/08/26 20:06:49 Oh. I see. I agree, since we don't show the anonym
// If we're at rest, hide the shadow. The Android view should be drawing.
toolbar_layer_->layer()->SetMasksToBounds(top_offset >= 0.f

Powered by Google App Engine
This is Rietveld 408576698