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

chrome/browser/renderer_host/render_widget_host_view.h

Index: chrome/browser/renderer_host/render_widget_host_view.h
diff --git a/chrome/browser/renderer_host/render_widget_host_view.h b/chrome/browser/renderer_host/render_widget_host_view.h
index 81de5060842a36195d24403051900cf1b35f22c2..c057b91027def3534461f44573ef2176030f5ea1 100644
--- a/chrome/browser/renderer_host/render_widget_host_view.h
+++ b/chrome/browser/renderer_host/render_widget_host_view.h
@@ -89,6 +89,9 @@ class RenderWidgetHostView {
   // Tells the View to size itself to the specified size.
   virtual void SetSize(const gfx::Size& size) = 0;
 
+   // Tells the View to move itself to the specified point.
+  virtual void SetMove(const gfx::Point& origin) = 0;

[Ilya Sherman, 2010/12/22 01:16:35]

nit: MoveTo() might be a better name for this method

[Ilya Sherman, 2010/12/22 01:16:35]

If you add this method to RenderWidgetHostView, you'll need to add
implementations for all subclasses, not just the Linux one.

[honten.org, 2010/12/22 01:29:41]

Yes, you are right.

So once this solution is Ok only in Linux, I'll start implement Mac.

But, as I used to say, I don't have any Windows dev env.
So I might ask somebody to implement Win.

On 2010/12/22 01:16:35, Ilya Sherman wrote:

+
   // Retrieves the native view used to contain plugins and identify the
   // renderer in IPC messages.
   virtual gfx::NativeView GetNativeView() = 0;