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

Unified Diff: Source/web/WebViewImpl.cpp

Issue 889803004: Add touchpad pinch zoom support to Blink (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Update tests Created 5 years, 11 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: Source/web/WebViewImpl.cpp
diff --git a/Source/web/WebViewImpl.cpp b/Source/web/WebViewImpl.cpp
index 06e1c3efd3a9d6a137123b869811287b1502407b..cf1eb22809e8dfedaaa1ec0ad05a6268ba0cf9d3 100644
--- a/Source/web/WebViewImpl.cpp
+++ b/Source/web/WebViewImpl.cpp
@@ -2140,8 +2140,20 @@ bool WebViewImpl::handleInputEvent(const WebInputEvent& inputEvent)
// Unhandled touchpad gesture pinch events synthesize mouse wheel events.
if (inputEvent.type == WebInputEvent::GesturePinchUpdate) {
- if (handleSyntheticWheelFromTouchpadPinchEvent(static_cast<const WebGestureEvent&>(inputEvent)))
+ const WebGestureEvent& pinchEvent = static_cast<const WebGestureEvent&>(inputEvent);
+
+ // First, synthesize a Windows-like wheel event to send to any handlers that may exist.
+ if (handleSyntheticWheelFromTouchpadPinchEvent(pinchEvent))
+ return true;
+
+ if (pinchVirtualViewportEnabled()
+ && page()->frameHost().pinchViewport().magnifyScaleAroundAnchor(pinchEvent.data.pinchUpdate.scale, FloatPoint(pinchEvent.x, pinchEvent.y)))
return true;
+
+ // TODO(ccameron): Always return that the event was handled. Once the pinch-zoom content scale
+ // handlers have been removed from content, this function may return false if the page scale does
+ // not change.
+ return true;
}
return false;
« LayoutTests/fast/events/gesture-pinch-fake-mouse-wheel.html ('K') | « Source/web/WebViewImpl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698