diff options
| author | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-28 18:36:37 +0000 |
|---|---|---|
| committer | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-28 18:36:37 +0000 |
| commit | d454745055f7ec8c6b5fc158c61f66452d39aabf (patch) | |
| tree | 73395f7721bd5a23886a30afcab4ece31d88d866 /webkit/glue/chrome_client_impl.cc | |
| parent | 917543feedad73ad36a410413f9a5edc8b2e2c27 (diff) | |
| download | chromium_src-d454745055f7ec8c6b5fc158c61f66452d39aabf.zip chromium_src-d454745055f7ec8c6b5fc158c61f66452d39aabf.tar.gz chromium_src-d454745055f7ec8c6b5fc158c61f66452d39aabf.tar.bz2 | |
The GetWindowRect must return the rect of the container HWND; not the actual window rect. The usage of GetWindowRect in the ChromeClientImpl::windowRect() function is erroneous, and my modification of GetWindowRect to return the HWND's root ancestor rect broke drop down combo boxes and other embeded controls.
So instead, add a GetRootWindowRect which gets the root anncestor of the HWND's rect and use it in ChromeClientImpl::windowRect().
BUG=1344367,1186573,1334505
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1496 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/chrome_client_impl.cc')
| -rw-r--r-- | webkit/glue/chrome_client_impl.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/webkit/glue/chrome_client_impl.cc b/webkit/glue/chrome_client_impl.cc index 99b1882..3711abf 100644 --- a/webkit/glue/chrome_client_impl.cc +++ b/webkit/glue/chrome_client_impl.cc @@ -74,7 +74,7 @@ void ChromeClientImpl::setWindowRect(const WebCore::FloatRect& r) { WebCore::FloatRect ChromeClientImpl::windowRect() { if (webview_->delegate()) { gfx::Rect rect; - webview_->delegate()->GetWindowRect(webview_, &rect); + webview_->delegate()->GetRootWindowRect(webview_, &rect); return WebCore::FloatRect( static_cast<float>(rect.x()), static_cast<float>(rect.y()), |