nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1777936 - [bidi] script.evaluate awaitPromise should be mandatory r=webdriver-reviewers,whimboo

Browse source 
Differential Revision: https://phabricator.services.mozilla.com/D150920
This commit is contained in:
Julian Descottes 2022-07-05 07:11:41 +00:00
parent 1e71371aad
commit 26cb8439ba
1 changed files with 3 additions and 4 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

7
remote/webdriver-bidi/modules/root/script.jsm
View file

@ -126,10 +126,9 @@ class ScriptModule extends Module {
* realm or a browsing context. * realm or a browsing context.
* *
* @param {Object=} options * @param {Object=} options
* @param {boolean=} awaitPromise * @param {boolean} awaitPromise
* Determines if the command should wait for the return value of the * Determines if the command should wait for the return value of the
* expression to resolve, if this return value is a Promise. Defaults to * expression to resolve, if this return value is a Promise.
* true.
* @param {string} expression * @param {string} expression
* The expression to evaluate. * The expression to evaluate.
* @param {OwnershipModel=} resultOwnership [unsupported] * @param {OwnershipModel=} resultOwnership [unsupported]
@ -147,7 +146,7 @@ class ScriptModule extends Module {
*/ */
async evaluate(options = {}) { async evaluate(options = {}) {
const { const {
awaitPromise = true, awaitPromise,
expression: source, expression: source,
resultOwnership = OwnershipModel.None, resultOwnership = OwnershipModel.None,
target = {}, target = {},