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

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 185653013: [#8] Pass gfx::Point by const ref. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed build errors on other platforms Created 6 years, 10 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
« no previous file with comments | « cc/trees/layer_tree_host_impl.h ('k') | content/renderer/input/input_handler_proxy_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 178964000e743a3598655b8fc3f416b42274d8d5..e22ed102d7eafd7d11bb52033b466feee7481ffa 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -476,7 +476,8 @@ void LayerTreeHostImpl::ScheduleAnimation() {
SetNeedsRedraw();
}
-bool LayerTreeHostImpl::HaveTouchEventHandlersAt(gfx::Point viewport_point) {
+bool LayerTreeHostImpl::HaveTouchEventHandlersAt(
+ const gfx::Point& viewport_point) {
if (!settings_.touch_hit_testing)
return true;
if (!EnsureRenderSurfaceLayerList())
@@ -2069,7 +2070,8 @@ LayerImpl* LayerTreeHostImpl::FindScrollLayerForDeviceViewportPoint(
}
InputHandler::ScrollStatus LayerTreeHostImpl::ScrollBegin(
- gfx::Point viewport_point, InputHandler::ScrollInputType type) {
+ const gfx::Point& viewport_point,
+ InputHandler::ScrollInputType type) {
TRACE_EVENT0("cc", "LayerTreeHostImpl::ScrollBegin");
if (top_controls_manager_)
@@ -2197,7 +2199,7 @@ static gfx::Vector2dF ScrollLayerWithLocalDelta(LayerImpl* layer_impl,
return layer_impl->ScrollDelta() - previous_delta;
}
-bool LayerTreeHostImpl::ScrollBy(gfx::Point viewport_point,
+bool LayerTreeHostImpl::ScrollBy(const gfx::Point& viewport_point,
const gfx::Vector2dF& scroll_delta) {
TRACE_EVENT0("cc", "LayerTreeHostImpl::ScrollBy");
if (!CurrentlyScrollingLayer())
@@ -2326,7 +2328,7 @@ bool LayerTreeHostImpl::ScrollBy(gfx::Point viewport_point,
// This implements scrolling by page as described here:
// http://msdn.microsoft.com/en-us/library/windows/desktop/ms645601(v=vs.85).aspx#_win32_The_Mouse_Wheel
// for events with WHEEL_PAGESCROLL set.
-bool LayerTreeHostImpl::ScrollVerticallyByPage(gfx::Point viewport_point,
+bool LayerTreeHostImpl::ScrollVerticallyByPage(const gfx::Point& viewport_point,
ScrollDirection direction) {
DCHECK(wheel_scrolling_);
@@ -2433,7 +2435,7 @@ float LayerTreeHostImpl::DeviceSpaceDistanceToLayer(
device_viewport_point);
}
-void LayerTreeHostImpl::MouseMoveAt(gfx::Point viewport_point) {
+void LayerTreeHostImpl::MouseMoveAt(const gfx::Point& viewport_point) {
if (!EnsureRenderSurfaceLayerList())
return;
@@ -2531,7 +2533,7 @@ void LayerTreeHostImpl::PinchGestureBegin() {
}
void LayerTreeHostImpl::PinchGestureUpdate(float magnify_delta,
- gfx::Point anchor) {
+ const gfx::Point& anchor) {
if (!InnerViewportScrollLayer())
return;
« no previous file with comments | « cc/trees/layer_tree_host_impl.h ('k') | content/renderer/input/input_handler_proxy_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698