Cleanup ApplyViewportDeltas
This methods takes many arguments, with more to come, and is plumbed
into many places throughout the code base. This requires tons of boiler
plate for small changes.
This CL packages the args up into an "Args" struct. I've also changed
the term "Deltas" into "Changes" since not all the arguments are/will-be
"deltas" (in the SyncedProperty sense). Finally, the
outer_viewport_delta is no longer used but wasn't removed (due to the
above problem of plumbing) so I've done that here as well.
Bug: NONE
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I675f76bcf6c011bfce464d95bec746b580b37df2
Reviewed-on: https://chromium-review.googlesource.com/c/1267820
Reviewed-by: enne <[email protected]>
Reviewed-by: Antoine Labour <[email protected]>
Reviewed-by: Philip Rogers <[email protected]>
Commit-Queue: David Bokan <[email protected]>
Cr-Commit-Position: refs/heads/master@{#597693}diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index a24f837..88edfef 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -929,17 +929,11 @@
///////////////////////////////////////////////////////////////////////////////
// LayerTreeViewDelegate
-void RenderWidget::ApplyViewportDeltas(
- const gfx::Vector2dF& inner_delta,
- const gfx::Vector2dF& outer_delta,
- const gfx::Vector2dF& elastic_overscroll_delta,
- float page_scale,
- float top_controls_delta) {
+void RenderWidget::ApplyViewportChanges(
+ const cc::ApplyViewportChangesArgs& args) {
if (!GetWebWidget())
return;
- GetWebWidget()->ApplyViewportDeltas(inner_delta, outer_delta,
- elastic_overscroll_delta, page_scale,
- top_controls_delta);
+ GetWebWidget()->ApplyViewportChanges(args);
}
void RenderWidget::RecordWheelAndTouchScrollingCount(
