Use the pinch viewport offset for tap disambiguation.

public/web/WebViewClient.h

Index: public/web/WebViewClient.h
diff --git a/public/web/WebViewClient.h b/public/web/WebViewClient.h
index bc0bb7edf1af646efd89055f61af4171e0399921..fadc699595ad56a192d74838a3a6f187900a8972 100644
--- a/public/web/WebViewClient.h
+++ b/public/web/WebViewClient.h
@@ -204,9 +204,15 @@ public:
     // unless the view did not need a layout.
     virtual void didUpdateLayout() { }
 
+    // TODO: This is a deprecated method. Remove it after the chromium change
+    // that uses the second didTapMultipleTargets() propagates.
+    //
     // Return true to swallow the input event if the embedder will start a disambiguation popup
     virtual bool didTapMultipleTargets(const WebGestureEvent&, const WebVector<WebRect>& targetRects) { return false; }

[bokan, 2014/09/23 15:38:22]

I think the style is to keep param names unless they're easily implied from the
method name.

[timav, 2014/09/23 22:30:07]

Done.

 
+    // Return true to swallow the input event if the embedder will start a disambiguation popup
+    virtual bool didTapMultipleTargets(const WebSize&, const WebRect&, const WebVector<WebRect>& targetRects) { return false; }
+
     // Returns comma separated list of accept languages.
     virtual WebString acceptLanguages() { return WebString(); }