Title: [164488] trunk/Source/WebKit2
Revision
164488
Author
[email protected]
Date
2014-02-21 11:28:15 -0800 (Fri, 21 Feb 2014)

Log Message

Make sure to invoke the completion handler for JS alerts
https://bugs.webkit.org/show_bug.cgi?id=129158

Reviewed by Alexey Proskuryakov.

* UIProcess/API/C/WKPage.cpp:
(WKPageSetPageUIClient):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (164487 => 164488)


--- trunk/Source/WebKit2/ChangeLog	2014-02-21 19:17:28 UTC (rev 164487)
+++ trunk/Source/WebKit2/ChangeLog	2014-02-21 19:28:15 UTC (rev 164488)
@@ -1,3 +1,13 @@
+2014-02-21  Anders Carlsson  <[email protected]>
+
+        Make sure to invoke the completion handler for JS alerts
+        https://bugs.webkit.org/show_bug.cgi?id=129158
+
+        Reviewed by Alexey Proskuryakov.
+
+        * UIProcess/API/C/WKPage.cpp:
+        (WKPageSetPageUIClient):
+
 2014-02-20  Zalan Bujtas  <[email protected]>
 
         Remove redundant deviceScaleFactor() functions and make callers use Document::deviceScaleFactor() when accessible.

Modified: trunk/Source/WebKit2/UIProcess/API/C/WKPage.cpp (164487 => 164488)


--- trunk/Source/WebKit2/UIProcess/API/C/WKPage.cpp	2014-02-21 19:17:28 UTC (rev 164487)
+++ trunk/Source/WebKit2/UIProcess/API/C/WKPage.cpp	2014-02-21 19:28:15 UTC (rev 164488)
@@ -1167,6 +1167,7 @@
             }
 
             m_client.runJavaScriptAlert(toAPI(page), toAPI(message.impl()), toAPI(frame), m_client.base.clientInfo);
+            completionHandler();
         }
 
         virtual void runJavaScriptConfirm(WebPageProxy* page, const String& message, WebFrameProxy* frame, std::function<void (bool)> completionHandler) override
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to