REGRESSION(r244230): Web Automation: use null instead of empty string as success...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 May 2019 01:06:12 +0000 (01:06 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 May 2019 01:06:12 +0000 (01:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=197655

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2019-05-07
Reviewed by Devin Rousso.

This regressed in r244230, the case of an alert open while running a script is now handled in WebKit side, but
an empty string is returned as a result, which is not a valid JSON serialized string. That made all user prompts
tests to fail.

* UIProcess/Automation/WebAutomationSession.cpp:
(WebKit::WebAutomationSession::willShowJavaScriptDialog): Use null as script result instead of an empty string.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/Automation/WebAutomationSession.cpp

index b1f6771..009e11e 100644 (file)
@@ -1,3 +1,17 @@
+2019-05-07  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        REGRESSION(r244230): Web Automation: use null instead of empty string as success result of scripts when alert is open
+        https://bugs.webkit.org/show_bug.cgi?id=197655
+
+        Reviewed by Devin Rousso.
+
+        This regressed in r244230, the case of an alert open while running a script is now handled in WebKit side, but
+        an empty string is returned as a result, which is not a valid JSON serialized string. That made all user prompts
+        tests to fail.
+
+        * UIProcess/Automation/WebAutomationSession.cpp:
+        (WebKit::WebAutomationSession::willShowJavaScriptDialog): Use null as script result instead of an empty string.
+
 2019-05-07  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r245038.
index 018bf91..c25b951 100644 (file)
@@ -654,7 +654,7 @@ void WebAutomationSession::willShowJavaScriptDialog(WebPageProxy& page)
         if (!m_evaluateJavaScriptFunctionCallbacks.isEmpty()) {
             for (auto key : copyToVector(m_evaluateJavaScriptFunctionCallbacks.keys())) {
                 auto callback = m_evaluateJavaScriptFunctionCallbacks.take(key);
-                callback->sendSuccess(emptyString());
+                callback->sendSuccess("null"_s);
             }
         }