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

content/browser/renderer_host/render_widget_host.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/renderer_host/render_widget_host.h"
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
@@ skipping 780 matching lines @@
   // Is that really what we want in response to this message?  I'm matching
   // previous behavior of the code here.
   Destroy();
 }
 
 void RenderWidgetHost::OnMsgClose() {
   Shutdown();
 }
 
 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_));
   }
 }
 
 void RenderWidgetHost::OnMsgPaintAtSizeAck(int tag, const gfx::Size& size) {
   PaintAtSizeAckDetails details = {tag, size};
   gfx::Size size_details = size;
   NotificationService::current()->Notify(
       NotificationType::RENDER_WIDGET_HOST_DID_RECEIVE_PAINT_AT_SIZE_ACK,
       Source<RenderWidgetHost>(this),
@@ skipping 464 matching lines @@
     return;
 
   for (int i = 0; i < static_cast<int>(deferred_plugin_handles_.size()); i++) {
 #if defined(TOOLKIT_USES_GTK)
     view_->CreatePluginContainer(deferred_plugin_handles_[i]);
 #endif
   }
 
   deferred_plugin_handles_.clear();
 }