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

chrome/browser/renderer_host/render_widget_host_view_mac.mm

 // 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 <QuartzCore/QuartzCore.h>
 
 #include "chrome/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include "app/app_switches.h"
 #include "app/surface/io_surface_support_mac.h"
@@ skipping 601 matching lines @@
   // in a TabContentsViewCocoa, they're both BaseViews.
   DCHECK([[cocoa_view_ superview] isKindOfClass:[BaseView class]]);
   gfx::Rect rect =
       [(BaseView*)[cocoa_view_ superview] flipNSRectToRect:[cocoa_view_ frame]];
   rect.set_width(size.width());
   rect.set_height(size.height());
   [cocoa_view_ setFrame:
       [(BaseView*)[cocoa_view_ superview] flipRectToNSRect:rect]];
 }
 
+void RenderWidgetHostViewMac::MoveTo(const gfx::Point& point) {
+  if (is_hidden_)
+    return;
+
+  // Do conversions to upper-left origin, as "move to" means keep the
+  // upper-left corner pinned.

[Avi (use Gerrit), 2011/01/25 22:52:07]

You can't really say "pinned" here as that implies "not moving" but this is a
move call. More like, "Do conversions to upper-left origin, as the call
specifies the upper-left point."

+  DCHECK([[cocoa_view_ superview] isKindOfClass:[BaseView class]]);
+  gfx::Rect rect =
+      [(BaseView*)[cocoa_view_ superview] flipNSRectToRect:[cocoa_view_ frame]];
+  rect.set_x(point.x());
+  rect.set_y(point.y());
+  [cocoa_view_ setFrame:
+      [(BaseView*)[cocoa_view_ superview] flipRectToNSRect:rect]];
+}
+
 gfx::NativeView RenderWidgetHostViewMac::GetNativeView() {
   return native_view();
 }
 
 void RenderWidgetHostViewMac::MovePluginWindows(
     const std::vector<webkit::npapi::WebPluginGeometry>& moves) {
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   // Handle movement of accelerated plugins, which are the only "windowed"
   // plugins that exist on the Mac.
   for (std::vector<webkit::npapi::WebPluginGeometry>::const_iterator iter =
@@ skipping 2113 matching lines @@
   if (!string) return NO;
 
   // If the user is currently using an IME, confirm the IME input,
   // and then insert the text from the service, the same as TextEdit and Safari.
   [self confirmComposition];
   [self insertText:string];
   return YES;
 }
 
 @end