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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/input/synthetic_gesture_target_base.cc
diff --git a/content/browser/renderer_host/input/synthetic_gesture_target_base.cc b/content/browser/renderer_host/input/synthetic_gesture_target_base.cc
index 987dff4f7c84ff8a896ae5fa31aef858baf47064..e1d13d18a6d1c030d619803f43ef2ce13c452b01 100644
--- a/content/browser/renderer_host/input/synthetic_gesture_target_base.cc
+++ b/content/browser/renderer_host/input/synthetic_gesture_target_base.cc
@@ -121,6 +121,13 @@ int SyntheticGestureTargetBase::GetTouchSlopInDips() const {
return kTouchSlopInDips;
}
+int SyntheticGestureTargetBase::GetMinScalingSpanInDips() const {
+ // The minimum scaling distance is only relevant for touch gestures and the
+ // base target doesn't support touch.
+ 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.
+ return -1;
+}
+
bool SyntheticGestureTargetBase::PointIsWithinContents(int x, int y) const {
gfx::Rect bounds = host_->GetView()->GetViewBounds();
bounds -= bounds.OffsetFromOrigin(); // Translate the bounds to (0,0).

Powered by Google App Engine
This is Rietveld 408576698