GTK: Fix popup menu positioning....

chrome/browser/gtk/menu_gtk.cc

Index: chrome/browser/gtk/menu_gtk.cc
===================================================================
--- chrome/browser/gtk/menu_gtk.cc	(revision 32394)
+++ chrome/browser/gtk/menu_gtk.cc	(working copy)
@@ -322,6 +322,14 @@
   gfx::Point* point = reinterpret_cast<gfx::Point*>(userdata);
   *x = point->x();
   *y = point->y();
+
+  GtkRequisition menu_req;
+  gtk_widget_size_request(GTK_WIDGET(menu), &menu_req);
+  GdkScreen* screen = gdk_screen_get_default();

[Evan Martin, 2009/11/19 00:40:32]

Isn't this wrong if the menu's on a different screen?

[Evan Stade, 2009/11/19 00:46:18]

oh, like a dual monitor setup? didn't think about that. Guess I should just copy
the above code (or factor it out).

+  gint screen_height = gdk_screen_get_height(screen);
+
+  if (*y + menu_req.height > screen_height)
+    *y -= menu_req.height;
 }
 
 void MenuGtk::UpdateMenu() {