Look Up on Force Touch

third_party/WebKit/public/web/WebWidget.h

Index: third_party/WebKit/public/web/WebWidget.h
diff --git a/third_party/WebKit/public/web/WebWidget.h b/third_party/WebKit/public/web/WebWidget.h
index ceb006fab242ba7440712515e0060f2649f0d4de..b0064502e37893cafdf847970f46539b95830324 100644
--- a/third_party/WebKit/public/web/WebWidget.h
+++ b/third_party/WebKit/public/web/WebWidget.h
@@ -125,6 +125,11 @@ public:
     // Called to inform the WebWidget of an input event.
     virtual WebInputEventResult handleInputEvent(const WebInputEvent&) { return WebInputEventResult::NotHandled; }
 
+    virtual bool selectWordIfAnyAtCoordinates(float x, float y)

[Rick Byers, 2015/12/21 16:31:09]

Looking at this in more detail, it seems that it probably fits better into
WebView than WebWidget - would you mind moving it?  Eg. it seems to fit with the
other "data exchange" methods like "saveImageAt".

[Rick Byers, 2015/12/21 16:31:09]

For consistency with similar APIs, please use WebPoint as the argument type
instead of taking two bare floats, and then name it something like
"selectWordIfAnyAt".

WebFloatPoint is also an option, but mouse input is currently handled in
integers (and it looks like you're converting back and forth between floats and
ints) so WebPoint is probably better.

[Rick Byers, 2015/12/21 16:31:09]

please add a short description (we try to keep the blink public API pretty well
documented).  For example, you need to say what the return value indicates.

[spqchan, 2016/01/06 22:52:31]

Done, switched to WebPoint

[spqchan, 2016/01/06 22:52:31]

Sure thing. I also moved the code from render_widget to render_view_impl in
addition to this.

[spqchan, 2016/01/06 22:52:31]

Done.

+    {
+        return false;
+    }
+
     // Called to inform the WebWidget of the mouse cursor's visibility.
     virtual void setCursorVisibilityState(bool isVisible) { }