diff options
| author | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-18 08:38:13 +0000 |
|---|---|---|
| committer | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-18 08:38:13 +0000 |
| commit | dc514fa1a418fc28c847c52544165ba88bdd0c67 (patch) | |
| tree | c6b43b70afd44894a76cbaca35dc5b5b2830c13d | |
| parent | 4f166150b374f1fbaaf15c25d942458695bf340c (diff) | |
| download | chromium_src-dc514fa1a418fc28c847c52544165ba88bdd0c67.zip chromium_src-dc514fa1a418fc28c847c52544165ba88bdd0c67.tar.gz chromium_src-dc514fa1a418fc28c847c52544165ba88bdd0c67.tar.bz2 | |
Fix for Issue 7651 Regression: The position of input method window is not correct after scrolling page down.
This is caused by my bonehead mistake in fixing Issue 6652 that I forgot calling the FrameView::contentsToWindow() function. (I need to add a unit-test to prevent this regression, though.)
BUG=7651
Review URL: http://codereview.chromium.org/21338
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9942 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | webkit/glue/webview_impl.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/webkit/glue/webview_impl.cc b/webkit/glue/webview_impl.cc index 0259b72..7959b6f 100644 --- a/webkit/glue/webview_impl.cc +++ b/webkit/glue/webview_impl.cc @@ -1185,7 +1185,10 @@ bool WebViewImpl::ImeUpdateStatus(bool* enable_ime, return false; *enable_ime = node->shouldUseInputMethod() && !controller->isInPasswordField(); - const IntRect rect(controller->absoluteCaretBounds()); + const FrameView* view = node->document()->view(); + if (!view) + return false; + const IntRect rect(view->contentsToWindow(controller->absoluteCaretBounds())); caret_rect->SetRect(rect.x(), rect.y(), rect.width(), rect.height()); return true; } |