Changeset 99589 in webkit

Timestamp:

Nov 8, 2011 10:23:30 AM (12 years ago)

Author:

commit-queue@webkit.org

Message:

Unreviewed, rolling out r95720.
​http://trac.webkit.org/changeset/95720
​https://bugs.webkit.org/show_bug.cgi?id=71831

This has made the menulists on WebKit patch reviews much less
usable (Requested by mrobinson on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-11-08

• platform/gtk/GtkPopupMenu.cpp:

(WebCore::GtkPopupMenu::menuPositionFunction):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/gtk/GtkPopupMenu.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-11-08  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r95720.
+        http://trac.webkit.org/changeset/95720
+        https://bugs.webkit.org/show_bug.cgi?id=71831
+
+        This has made the menulists on WebKit patch reviews much less
+        usable (Requested by mrobinson on #webkit).
+
+        * platform/gtk/GtkPopupMenu.cpp:
+        (WebCore::GtkPopupMenu::menuPositionFunction):
+
 2011-11-08  Nico Weber  <thakis@chromium.org>
 

trunk/Source/WebCore/platform/gtk/GtkPopupMenu.cpp

@@ -7 +7 @@
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  * Copyright (C) 2010-2011 Igalia S.L.
- * Portions Copyright (c) 2010 Motorola Mobility, Inc.  All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ -147 +146 @@
     *x = popupMenu->m_menuPosition.x();
     *y = popupMenu->m_menuPosition.y();
-    *pushIn = false;
+    *pushIn = true;
 }