What's the difference to bug 1264259?

Wait. Drop my last comment. I missed to read the full bug summary.

WebDriver:{ExecuteScript,ExecuteAsyncScript} were missing user prompt checks. This patch does not add any tests because Marionette does not yet implement the user prompt handler as described in the specification, which means that the relevant tests in WPT are incorrect for this behaviour. Review commit: https://reviewboard.mozilla.org/r/222316/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/222316/

WebDriver:{ExecuteScript,ExecuteAsyncScript} were missing user prompt checks. Review commit: https://reviewboard.mozilla.org/r/259792/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/259792/

To align with all the other commands the user_prompts.py test file has to contain tests for handling an open user prompt when calling the command. In case of both Execute Script and Execute Async Script the file contained tests where a user prompt got opened during the script execution. Tests for the latter situation are moved to a different test file. Review commit: https://reviewboard.mozilla.org/r/259794/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/259794/

Comment on attachment 8995334 [details] Bug 1439995 - [marionette] Add user prompt checks for script execution commands. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/259792/diff/1-2/

Comment on attachment 8995335 [details] Bug 1439995 - [wdspec] Refactor user_prompt tests for Execute Script / Execute Async Script. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/259794/diff/1-2/