From bd65bb67e25a79b019d745b7262b2008ce2adb15 Mon Sep 17 00:00:00 2001 From: Florian Dold Date: Wed, 16 Nov 2016 01:59:39 +0100 Subject: incrementally verify denoms The denominations are not stored in a separate object store. --- node_modules/selenium-webdriver/remote/index.js | 28 ++++++++++++++----------- 1 file changed, 16 insertions(+), 12 deletions(-) (limited to 'node_modules/selenium-webdriver/remote/index.js') diff --git a/node_modules/selenium-webdriver/remote/index.js b/node_modules/selenium-webdriver/remote/index.js index ab76b4476..87dee7e8d 100644 --- a/node_modules/selenium-webdriver/remote/index.js +++ b/node_modules/selenium-webdriver/remote/index.js @@ -247,15 +247,18 @@ class DriverService { pathname: self.path_ }); - return new Promise(function(fulfill, reject) { - var ready = httpUtil.waitForServer(serverUrl, timeout) - .then(fulfill, reject); - earlyTermination.catch(function(e) { - ready.cancel(/** @type {Error} */(e)); - reject(Error(e.message)); - }); - }).then(function() { - return serverUrl; + return new Promise((fulfill, reject) => { + let cancelToken = + earlyTermination.catch(e => reject(Error(e.message))); + + httpUtil.waitForServer(serverUrl, timeout, cancelToken) + .then(_ => fulfill(serverUrl), err => { + if (err instanceof promise.CancellationError) { + fulfill(serverUrl); + } else { + reject(err); + } + }); }); }); })); @@ -283,7 +286,7 @@ class DriverService { /** * Schedules a task in the current control flow to stop the server if it is * currently running. - * @return {!promise.Promise} A promise that will be resolved when + * @return {!promise.Thenable} A promise that will be resolved when * the server has been stopped. */ stop() { @@ -297,8 +300,9 @@ class DriverService { * @return {!Promise>} */ function resolveCommandLineFlags(args) { - return Promise.resolve(args) // Resolve the outer array. - .then(args => Promise.all(args)); // Then resolve the individual flags. + // Resolve the outer array, then the individual flags. + return Promise.resolve(args) + .then(/** !Array */args => Promise.all(args)); } -- cgit v1.2.3