diff options
| author | vandebo@chromium.org <vandebo@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-19 23:10:08 +0000 |
|---|---|---|
| committer | vandebo@chromium.org <vandebo@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-19 23:10:08 +0000 |
| commit | 905287e26812ce758d112c556c40d698e1e063c2 (patch) | |
| tree | 42fddb9240341d8c854d13e42f935de40262c8a3 /webkit/glue | |
| parent | 26d1c58cbff72cdf752a78c1cf64a24728aba7ab (diff) | |
| download | chromium_src-905287e26812ce758d112c556c40d698e1e063c2.zip chromium_src-905287e26812ce758d112c556c40d698e1e063c2.tar.gz chromium_src-905287e26812ce758d112c556c40d698e1e063c2.tar.bz2 | |
Neither of the hit tests in WebViewImpl worked if the page had scrolled. This makes it work.
BUG=25261, 22810
TEST=Scroll the page down and test functionality...
Review URL: http://codereview.chromium.org/305003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29474 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue')
| -rw-r--r-- | webkit/glue/webview_impl.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/webkit/glue/webview_impl.cc b/webkit/glue/webview_impl.cc index e0340bd..d8654b0 100644 --- a/webkit/glue/webview_impl.cc +++ b/webkit/glue/webview_impl.cc @@ -489,6 +489,7 @@ void WebViewImpl::MouseDown(const WebMouseEvent& event) { if (focused_node.get() && webkit_glue::NodeToHTMLInputElement(focused_node.get())) { IntPoint point(event.x, event.y); + point = page_->mainFrame()->view()->windowToContents(point); HitTestResult result(point); result = page_->mainFrame()->eventHandler()->hitTestResultAtPoint(point, false); @@ -575,9 +576,10 @@ void WebViewImpl::MouseUp(const WebMouseEvent& event) { if (event.button == WebMouseEvent::ButtonMiddle) { Frame* focused = GetFocusedWebCoreFrame(); IntPoint click_point(last_mouse_down_point_.x, last_mouse_down_point_.y); + click_point = page_->mainFrame()->view()->windowToContents(click_point); HitTestResult hit_test_result = - focused->eventHandler()->hitTestResultAtPoint(click_point, false, false, - ShouldHitTestScrollbars); + focused->eventHandler()->hitTestResultAtPoint(click_point, false, false, + ShouldHitTestScrollbars); // We don't want to send a paste when middle clicking a scroll bar or a // link (which will navigate later in the code). if (!hit_test_result.scrollbar() && !hit_test_result.isLiveLink() && |