Consider the popup window position when the window shows upward. This patch depends on WebKit patch.

content/browser/renderer_host/render_widget_host.cc

Index: content/browser/renderer_host/render_widget_host.cc
diff --git a/content/browser/renderer_host/render_widget_host.cc b/content/browser/renderer_host/render_widget_host.cc
index 076d3e04d56832242ce39a07c9c4ad44c288a04e..5ddd7e5b1bb370f6dab317b30710853c70da68e6 100644
--- a/content/browser/renderer_host/render_widget_host.cc
+++ b/content/browser/renderer_host/render_widget_host.cc
@@ -800,7 +800,7 @@ void RenderWidgetHost::OnMsgClose() {
 void RenderWidgetHost::OnMsgRequestMove(const gfx::Rect& pos) {
   // Note that we ignore the position.

[Peter Kasting, 2011/03/07 19:25:44]

This comment lies.

Is the change below desirable?

[honten.org, 2011/03/07 19:36:07]

You are right.

We should should remove it.

On 2011/03/07 19:25:44, Peter Kasting wrote:

[Peter Kasting, 2011/03/07 20:20:15]

I'm just really curious.  "Request move" sounds like something that ought to
respect position.  Why were we not respecting it before?  Why is it right to
ignore it now?

[honten.org, 2011/03/07 21:19:50]

I don't know why. AFAIK, |pos| originally doesn't have correct position
information. That has only size info.

I'm not sure who implemented first.

On 2011/03/07 20:20:15, Peter Kasting wrote:

   if (view_) {
-    view_->SetSize(pos.size());
+    view_->SetBounds(pos);
     Send(new ViewMsg_Move_ACK(routing_id_));
   }
 }