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

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 18202006: Make scrollable viewport size no longer depend on clip layer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Apply to MaxScrollOffset Created 7 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: cc/trees/layer_tree_host_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index 282471fdc7bcdc85a1506fe4f55bed5fc595c331..33e90783e3fd448e48e2a8230d0c9adc47f0d662 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -1291,13 +1291,6 @@ gfx::SizeF LayerTreeHostImpl::VisibleViewportSize() const {
gfx::SizeF dip_size =
gfx::ScaleSize(device_viewport_size(), 1.f / device_scale_factor());
- // The clip layer should be used if non-overlay scrollbars may exist since
- // it adjusts for them.
- LayerImpl* clip_layer = active_tree_->RootClipLayer();
- if (!Settings().solid_color_scrollbars && clip_layer &&
- clip_layer->masks_to_bounds())
- dip_size = clip_layer->bounds();
-
float top_offset =
top_controls_manager_ ? top_controls_manager_->content_top_offset() : 0.f;
return gfx::SizeF(dip_size.width(),

Powered by Google App Engine
This is Rietveld 408576698