Clean up on screen keyboard callback interface on Windows.
Change API callback interface to be in DIPs. It is much simplier to
implement that for InputPane.
Don't send the scroll into view message as that is already sent
with the SetInsets message and eliminates the query for the current
mouse cursor position to determine the touch position which is wrong.
Remove the occluded rect from the hidden callback as it isn't used.
BUG=817501
Change-Id: I94050ede776afcbb358d2a997ecf691bb8d1c78b
Reviewed-on: https://chromium-review.googlesource.com/949783
Commit-Queue: Dave Tapuska <[email protected]>
Reviewed-by: Scott Violet <[email protected]>
Cr-Commit-Position: refs/heads/master@{#541141}diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 44764ef9..13c64464 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -6093,16 +6093,10 @@
void WebContentsImpl::FocusedNodeTouched(bool editable) {
#if defined(OS_WIN)
- // We use the cursor position to determine where the touch occurred.
RenderWidgetHostView* view = GetRenderWidgetHostView();
if (!view)
return;
- POINT cursor_pos = {};
- ::GetCursorPos(&cursor_pos);
- float scale = GetScaleFactorForView(view);
- gfx::Point location_dips_screen =
- gfx::ConvertPointToDIP(scale, gfx::Point(cursor_pos));
- view->FocusedNodeTouched(location_dips_screen, editable);
+ view->FocusedNodeTouched(editable);
#endif
}
