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

Unified Diff: cc/trees/layer_tree_impl.cc

Issue 17550008: Make IsSolidColor() a property on CC scrollbar layers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Set color/thickness via command line also. Created 7 years, 6 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: cc/trees/layer_tree_impl.cc
diff --git a/cc/trees/layer_tree_impl.cc b/cc/trees/layer_tree_impl.cc
index 30616268f2053ee494aad578457546f0fa88c2d5..0ced77d57383df36d9ebb5ebc1256a259b50d82b 100644
--- a/cc/trees/layer_tree_impl.cc
+++ b/cc/trees/layer_tree_impl.cc
@@ -223,7 +223,13 @@ void LayerTreeImpl::UpdateMaxScrollOffset() {
}
void LayerTreeImpl::UpdateSolidColorScrollbars() {
- DCHECK(settings().solid_color_scrollbars);
+ // TODO(wjmaclean) This logic will need to change when we re-factor
+ // RootScrollLayer() into InnerViewportScrollLayer() and
+ // OuterViewportScrollLayer(), since in the desktop case the
+ // OuterViewportScrollLayer() won't have solid color
+ // scrollbars, while the InnerViewportScrollLayer() will. Ultimately, the
+ // plan is for this function to disappear altogether.
+ DCHECK(settings().force_solid_color_scrollbars);
LayerImpl* root_scroll = RootScrollLayer();
DCHECK(root_scroll);
@@ -252,7 +258,7 @@ void LayerTreeImpl::UpdateDrawProperties() {
if (IsActiveTree() && RootScrollLayer() && RootClipLayer())
UpdateRootScrollLayerSizeDelta();
- if (settings().solid_color_scrollbars &&
+ if (settings().force_solid_color_scrollbars &&
IsActiveTree() &&
RootScrollLayer()) {
UpdateSolidColorScrollbars();

Powered by Google App Engine
This is Rietveld 408576698