aura: Fix popup window positioning.

aura: Fix popup window positioning.

Comboboxes in a web-page show up in the wrong place if the browser window is not
positioned at (0,0).

BUG=104224
TEST=manually

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110000

[Ben Goodger (Google), 2011-11-14 23:36:17]

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
File content/browser/renderer_host/render_widget_host_view_aura.cc (right):

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_aura.cc:132: // |pos| is
in desktop coordinate. So convert it to
coordinates

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_aura.cc:133: //
|popup_parent_host_view_|'s coordinate first.
coordinates

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_aura.cc:136:
static_cast<aura::Window*>(aura::Desktop::GetInstance()),
cast isn't necessary?

[sadrul, 2011-11-14 23:42:51]

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
File content/browser/renderer_host/render_widget_host_view_aura.cc (right):

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_aura.cc:132: // |pos| is
in desktop coordinate. So convert it to
On 2011/11/14 23:36:17, Ben Goodger (Google) wrote:
> coordinates

Done.

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_aura.cc:133: //
|popup_parent_host_view_|'s coordinate first.
On 2011/11/14 23:36:17, Ben Goodger (Google) wrote:
> coordinates

Done.

http://codereview.chromium.org/8555020/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_aura.cc:136:
static_cast<aura::Window*>(aura::Desktop::GetInstance()),
On 2011/11/14 23:36:17, Ben Goodger (Google) wrote:
> cast isn't necessary?

Indeed. Removed.

[Ben Goodger (Google), 2011-11-14 23:46:06]

lgtm