Rename VisualViewport properties to match updated spec

third_party/WebKit/Source/core/frame/VisualViewport.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 199 matching lines @@
   GraphicsLayer* LayerForHorizontalScrollbar() const override;
   GraphicsLayer* LayerForVerticalScrollbar() const override;
   bool ScheduleAnimation() override;
   CompositorAnimationHost* GetCompositorAnimationHost() const override;
   CompositorAnimationTimeline* GetCompositorAnimationTimeline() const override;
   IntRect VisibleContentRect(
       IncludeScrollbarsInRect = kExcludeScrollbars) const override;
   RefPtr<WebTaskRunner> GetTimerTaskRunner() const final;
 
   // Visual Viewport API implementation.
-  double ScrollLeft();
-  double ScrollTop();
-  double ClientWidth();
-  double ClientHeight();
-  double PageScale();
+  double OffsetLeft();

[dtapuska, 2017/05/24 17:21:40]

Should these be const?

[bokan, 2017/05/24 19:17:14]

Done.

+  double OffsetTop();
+  double Width();
+  double Height();
+  double ScaleForVisualViewport();
 
   // Used for gathering data on user pinch-zoom statistics.
   void UserDidChangeScale();
   void SendUMAMetrics();
   void StartTrackingPinchStats();
 
   // Heuristic-based function for determining if we should disable workarounds
   // for viewing websites that are not optimized for mobile devices.
   bool ShouldDisableDesktopWorkarounds() const;
 
@@ skipping 48 matching lines @@
   float scale_;
   IntSize size_;
   float browser_controls_adjustment_;
   float max_page_scale_;
   bool track_pinch_zoom_stats_for_page_;
 };
 
 }  // namespace blink
 
 #endif  // VisualViewport_h