Add touchpad pinch zoom support to Blink

Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 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 2122 matching lines @@
             eventType, static_cast<const WebMouseEvent&>(inputEvent).clickCount);
         return true;
     }
 
     // FIXME: This should take in the intended frame, not the local frame root.
     if (PageWidgetDelegate::handleInputEvent(*this, inputEvent, localFrameRootTemporary()->frame()))
         return true;
 
     // 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 the event remains unhandled, use it pinch the viewport.
+        const float magnifyDelta = pinchEvent.data.pinchUpdate.scale;
+        const float oldPageScale = pageScaleFactor();
+        const float newPageScale = clampPageScaleFactorToLimits(magnifyDelta * oldPageScale);
+
+        // Keep the center-of-pinch anchor specified by (x, y) in a stable
+        // position over the course of the magnify.
+        const FloatPoint anchor(pinchEvent.x, pinchEvent.y);

[bokan, 2015/02/03 00:45:12]

Drive-by: Could you please put this logic inside PinchViewport in a method that
takes the magnifyDelta and anchor?

[ccameron, 2015/02/03 02:29:08]

I don't see where the scale factor limits are available in PinchViewport, so I
had the function take the new scale factor, and the caller compute the two (the
caller can early-out and not return true if they are different). LMK if I missed
a way to do this entirely in PinchViewport.

[aelias_OOO_until_Jul13, 2015/02/03 02:37:14]

You can plumb in the information from WebViewImpl by adding an extra method to
ChromeClientImpl (which you can see in this file is reachable from FrameHost). 
I think it would be nicer to have the PinchViewport method take a delta.

[bokan, 2015/02/03 03:05:16]

Right, sorry, forgot about that. You can take Alexandre's suggestion above. 

[ccameron, 2015/02/03 17:03:28]

Done (yeah, I prefer this too -- thanks for pointing out how we get back to
WebViewImpl from PinchViewport).

+        FloatPoint anchorAtOldScale = anchor.scaledBy(1.f / oldPageScale);
+        FloatPoint anchorAtNewScale = anchor.scaledBy(1.f / newPageScale);
+        FloatPoint newLocation(pinchViewportOffset() + anchorAtOldScale - anchorAtNewScale);
+        setPageScaleFactorAndLocation(newPageScale, newLocation);

[aelias_OOO_until_Jul13, 2015/02/03 00:16:58]

Sorry, I noticed another issue.  Please scroll the normal FrameView scroll
offset first, and only apply the unused portion to the location in
setPageScaleFactorAndLocation.  That will match the current logic in
LayerTreeHostImpl::PinchGestureUpdate.  That will allow pinch zooming out to not
get clamped in a weird way, among other things.

[bokan, 2015/02/03 00:45:12]

Though almost dead, please be aware of the old pinch-path and guard necessary
calls behind the pinchVirtualViewportEnabled() flag. As-is
setPageScaleFactorAndLocation will ASSERT in the old-path. I'll be removing it
by the end of this quarter.

pinchViewportOffset() will return 0 in the old path which I think is fine in
this case.

[ccameron, 2015/02/03 02:29:08]

Done (in the PinchVirtualViewport version).

+
+        // 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;
 }
 
 void WebViewImpl::setCursorVisibilityState(bool isVisible)
 {
     if (m_page)
         m_page->setIsCursorVisible(isVisible);
 }
@@ skipping 2451 matching lines @@
     return mainFrameImpl()->frameView()->layoutSize().width() == m_size.width
         || (constraints.minimumScale == constraints.maximumScale && constraints.minimumScale != -1);
 }
 
 void WebViewImpl::forceNextWebGLContextCreationToFail()
 {
     WebGLRenderingContext::forceNextWebGLContextCreationToFail();
 }
 
 } // namespace blink