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

Unified Diff: cc/trees/layer_tree_host_impl.h

Issue 476113004: Replace overdraw_bottom_height with top_controls_layout_height. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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: cc/trees/layer_tree_host_impl.h
diff --git a/cc/trees/layer_tree_host_impl.h b/cc/trees/layer_tree_host_impl.h
index 878747874c7298b93b554ab645f0c15fe1309dd5..3d43962b81240be6ef2895448885334d99a68535 100644
--- a/cc/trees/layer_tree_host_impl.h
+++ b/cc/trees/layer_tree_host_impl.h
@@ -227,12 +227,6 @@ class CC_EXPORT LayerTreeHostImpl
// Viewport rect in view space used for tiling prioritization.
const gfx::Rect ViewportRectForTilePriority() const;
- // Viewport size for scrolling and fixed-position compensation. This value
- // excludes the URL bar and non-overlay scrollbars and is in DIP (and
- // invariant relative to page scale).
- gfx::SizeF UnscaledScrollableViewportSize() const;
- float VerticalAdjust() const;
-
// RendererClient implementation.
virtual void SetFullRootLayerDamage() OVERRIDE;
virtual void RunOnDemandRasterTask(Task* on_demand_raster_task) OVERRIDE;
@@ -350,8 +344,10 @@ class CC_EXPORT LayerTreeHostImpl
void SetViewportSize(const gfx::Size& device_viewport_size);
gfx::Size device_viewport_size() const { return device_viewport_size_; }
- void SetOverdrawBottomHeight(float overdraw_bottom_height);
- float overdraw_bottom_height() const { return overdraw_bottom_height_; }
+ void SetTopControlsLayoutHeight(float top_controls_layout_height);
+ float top_controls_layout_height() const {
+ return top_controls_layout_height_;
+ }
void SetOverhangUIResource(UIResourceId overhang_ui_resource_id,
const gfx::Size& overhang_ui_resource_size);
@@ -499,7 +495,6 @@ class CC_EXPORT LayerTreeHostImpl
SharedBitmapManager* manager,
int id);
- gfx::SizeF ComputeInnerViewportContainerSize() const;
void UpdateInnerViewportContainerSize();
// Virtual for testing.
@@ -675,11 +670,8 @@ class CC_EXPORT LayerTreeHostImpl
UIResourceId overhang_ui_resource_id_;
gfx::Size overhang_ui_resource_size_;
- // Vertical amount of the viewport size that's known to covered by a
- // browser-side UI element, such as an on-screen-keyboard. This affects
- // scrollable size since we want to still be able to scroll to the bottom of
- // the page when the keyboard is up.
- float overdraw_bottom_height_;
+ // Height of the top controls as known by Blink.
+ float top_controls_layout_height_;
// Optional top-level constraints that can be set by the OutputSurface.
// - external_transform_ applies a transform above the root layer

Powered by Google App Engine
This is Rietveld 408576698