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

Side by Side Diff: content/browser/renderer_host/input/synthetic_gesture_target_base.cc

Issue 293683002: Synthetic pinch gesture take scale factor as parameter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/input/synthetic_gesture_target_base.h" 5 #include "content/browser/renderer_host/input/synthetic_gesture_target_base.h"
6 6
7 #include "content/browser/renderer_host/render_widget_host_impl.h" 7 #include "content/browser/renderer_host/render_widget_host_impl.h"
8 #include "content/browser/renderer_host/render_widget_host_view_base.h" 8 #include "content/browser/renderer_host/render_widget_host_view_base.h"
9 #include "content/browser/renderer_host/ui_events_helper.h" 9 #include "content/browser/renderer_host/ui_events_helper.h"
10 #include "content/common/input_messages.h" 10 #include "content/common/input_messages.h"
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 114
115 base::TimeDelta SyntheticGestureTargetBase::PointerAssumedStoppedTime() 115 base::TimeDelta SyntheticGestureTargetBase::PointerAssumedStoppedTime()
116 const { 116 const {
117 return base::TimeDelta::FromMilliseconds(kPointerAssumedStoppedTimeMs); 117 return base::TimeDelta::FromMilliseconds(kPointerAssumedStoppedTimeMs);
118 } 118 }
119 119
120 int SyntheticGestureTargetBase::GetTouchSlopInDips() const { 120 int SyntheticGestureTargetBase::GetTouchSlopInDips() const {
121 return kTouchSlopInDips; 121 return kTouchSlopInDips;
122 } 122 }
123 123
124 int SyntheticGestureTargetBase::GetMinScalingSpanInDips() const {
125 // The minimum scaling distance is only relevant for touch gestures and the
126 // base target doesn't support touch.
127 CHECK(false) << "Touch events not supported for this browser.";
Sami 2014/05/19 14:49:04 This could be a NOTREACHED() too. Maybe also leave
Dominik Grewe 2014/05/19 17:31:43 Done.
128 return -1;
129 }
130
124 bool SyntheticGestureTargetBase::PointIsWithinContents(int x, int y) const { 131 bool SyntheticGestureTargetBase::PointIsWithinContents(int x, int y) const {
125 gfx::Rect bounds = host_->GetView()->GetViewBounds(); 132 gfx::Rect bounds = host_->GetView()->GetViewBounds();
126 bounds -= bounds.OffsetFromOrigin(); // Translate the bounds to (0,0). 133 bounds -= bounds.OffsetFromOrigin(); // Translate the bounds to (0,0).
127 return bounds.Contains(x, y); 134 return bounds.Contains(x, y);
128 } 135 }
129 136
130 } // namespace content 137 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698