Unreviewed, rolling out r123463.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jul 2012 13:42:56 +0000 (13:42 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jul 2012 13:42:56 +0000 (13:42 +0000)
http://trac.webkit.org/changeset/123463
https://bugs.webkit.org/show_bug.cgi?id=92110

Broke Android build (Requested by keishi on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-07-24

* src/WebPagePopupImpl.h:
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::handleMouseDown):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@123470 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/WebPagePopupImpl.h
Source/WebKit/chromium/src/WebViewImpl.cpp

index 9d8f4cd..b585ee4 100644 (file)
@@ -1,3 +1,15 @@
+2012-07-24  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r123463.
+        http://trac.webkit.org/changeset/123463
+        https://bugs.webkit.org/show_bug.cgi?id=92110
+
+        Broke Android build (Requested by keishi on #webkit).
+
+        * src/WebPagePopupImpl.h:
+        * src/WebViewImpl.cpp:
+        (WebKit::WebViewImpl::handleMouseDown):
+
 2012-07-24  Keishi Hattori  <keishi@webkit.org>
 
         [Chromium] Page popup should close on mouse down
index 2552bac..2e1c8c3 100644 (file)
@@ -63,7 +63,6 @@ public:
     bool handleKeyEvent(const WebCore::PlatformKeyboardEvent&);
     void closePopup();
     WebWidgetClient* widgetClient() const { return m_widgetClient; }
-    bool hasSamePopupClient(WebPagePopupImpl* other) { return other && m_popupClient == other->m_popupClient; }
 
 private:
     // WebWidget functions
index 295b21e..6a253f8 100644 (file)
@@ -522,15 +522,13 @@ void WebViewImpl::handleMouseLeave(Frame& mainFrame, const WebMouseEvent& event)
 
 void WebViewImpl::handleMouseDown(Frame& mainFrame, const WebMouseEvent& event)
 {
-    // If there is a popup open, close it as the user is clicking on the page (outside of the
-    // popup). We also save it so we can prevent a click on an element from immediately
-    // reopening the same popup.
+    // If there is a select popup open, close it as the user is clicking on
+    // the page (outside of the popup).  We also save it so we can prevent a
+    // click on the select element from immediately reopening the popup.
     RefPtr<WebCore::PopupContainer> selectPopup;
-    RefPtr<WebPagePopupImpl> pagePopup;
     if (event.button == WebMouseEvent::ButtonLeft) {
         selectPopup = m_selectPopup;
-        pagePopup = m_pagePopup;
-        hidePopups();
+        hideSelectPopup();
         ASSERT(!m_selectPopup);
     }
 
@@ -557,12 +555,6 @@ void WebViewImpl::handleMouseDown(Frame& mainFrame, const WebMouseEvent& event)
         hideSelectPopup();
     }
 
-    if (m_pagePopup && pagePopup && m_pagePopup->hasSamePopupClient(pagePopup.get())) {
-        // That click triggered a page popup that is the same as the one we just closed.
-        // It needs to be closed.
-        closePagePopup(m_pagePopup.get());
-    }
-
     // Dispatch the contextmenu event regardless of if the click was swallowed.
     // On Windows, we handle it on mouse up, not down.
 #if OS(DARWIN)