aura: Fix popup window positioning.

content/browser/renderer_host/render_widget_host_view_aura.cc

Index: content/browser/renderer_host/render_widget_host_view_aura.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_aura.cc b/content/browser/renderer_host/render_widget_host_view_aura.cc
index 39ff64a863d11f5a0cad44a8cde2a32f3eba377a..f89fc2dc965fc5a227471b70925c85f19541767b 100644
--- a/content/browser/renderer_host/render_widget_host_view_aura.cc
+++ b/content/browser/renderer_host/render_widget_host_view_aura.cc
@@ -128,7 +128,14 @@ void RenderWidgetHostViewAura::InitAsPopup(
 
   window_->SetParent(NULL);
   Show();
-  SetBounds(pos);
+
+  // |pos| is in desktop coordinate. So convert it to

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

coordinates

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

Done.

+  // |popup_parent_host_view_|'s coordinate first.

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

coordinates

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

Done.

+  gfx::Point origin = pos.origin();
+  aura::Window::ConvertPointToWindow(
+      static_cast<aura::Window*>(aura::Desktop::GetInstance()),

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

cast isn't necessary?

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

Indeed. Removed.

+      popup_parent_host_view_->window_, &origin);
+  SetBounds(gfx::Rect(origin, pos.size()));
 }
 
 void RenderWidgetHostViewAura::InitAsFullscreen(