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

chrome/browser/renderer_host/render_widget_host_view_win.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_view_win.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/i18n/rtl.h"
@@ skipping 349 matching lines @@
   // reduce its resource utilization.
   render_widget_host_->WasHidden();
 
   // TODO(darin): what about constrained windows?  it doesn't look like they
   // see a message when their parent is hidden.  maybe there is something more
   // generic we can do at the TabContents API level instead of relying on
   // Windows messages.
 }
 
 void RenderWidgetHostViewWin::SetSize(const gfx::Size& size) {
+  gfx::Rect rect = GetViewBounds();
+  rect.set_size(size);
+  SetBounds(rect);
+}
+
+void RenderWidgetHostViewWin::SetBounds(const gfx::Rect& rect) {
   if (is_hidden_)
     return;
 
-  // No SWP_NOREDRAW as autofill popups can resize and the underneath window
+  // No SWP_NOREDRAW as autofill popups can move and the underneath window
   // should redraw in that case.
   UINT swp_flags = SWP_NOSENDCHANGING | SWP_NOOWNERZORDER | SWP_NOCOPYBITS |
-      SWP_NOMOVE | SWP_NOZORDER | SWP_NOACTIVATE | SWP_DEFERERASE;
-  SetWindowPos(NULL, 0, 0, size.width(), size.height(), swp_flags);
+      SWP_NOZORDER | SWP_NOACTIVATE | SWP_DEFERERASE;
+  gfx::Rect org_rect = GetViewBounds();
+  if (rect.origin() == org_rect.orgin())

[Peter Kasting, 2011/02/16 22:01:25]

Nit: I don't think you need this conditional.  SWP_NOMOVE is only needed if
you're going to pass "wrong" origin coordinates that need to be ignored.

+    swp_flags |= SWP_NOMOVE;
+  SetWindowPos(NULL, point.x(), point.y(), window_rect.Width(),

[Peter Kasting, 2011/02/16 22:01:25]

You need to change both |point| and |window_rect| to |rect|, and un-capitalize
Width() and Height(), here and below.

+               window_rect.Height(), swp_flags);
   if (compositor_host_window_) {
     ::SetWindowPos(compositor_host_window_,
         NULL,
-        0, 0,
-        size.width(), size.height(),
+        point.x(), point.y(),
+        rect.Width(), rect.Height(),
         SWP_NOSENDCHANGING | SWP_NOCOPYBITS | SWP_NOZORDER | SWP_NOACTIVATE);
   }
   render_widget_host_->WasResized();
   EnsureTooltip();
 }
 
 gfx::NativeView RenderWidgetHostViewWin::GetNativeView() {
   return m_hWnd;
 }
 
@@ skipping 1417 matching lines @@
 }
 
 // static
 RenderWidgetHostView*
     RenderWidgetHostView::GetRenderWidgetHostViewFromNativeView(
         gfx::NativeView native_view) {
   return ::IsWindow(native_view) ?
       reinterpret_cast<RenderWidgetHostView*>(
           ViewProp::GetValue(native_view, kRenderWidgetHostViewKey)) : NULL;
 }