selenium-webdriver 3.0.0-beta-1 → 3.0.1

package/http/util.js

@@ -61,38 +61,54 @@ exports.getStatus = getStatus;
  * Waits for a WebDriver server to be healthy and accepting requests.
  * @param {string} url Base URL of the server to query.
  * @param {number} timeout How long to wait for the server.
- * @return {!promise.Promise} A promise that will resolve when the
- *     server is ready.
+ * @param {Promise=} opt_cancelToken A promise used as a cancellation signal:
+ *     if resolved before the server is ready, the wait will be terminated
+ *     early with a {@link promise.CancellationError}.
+ * @return {!Promise} A promise that will resolve when the server is ready, or
+ *     if the wait is cancelled.
  */
-exports.waitForServer = function(url, timeout) {
-  var ready = promise.defer(),
-      start = Date.now();
-  checkServerStatus();
-  return ready.promise;
-
-  function checkServerStatus() {
-    return getStatus(url).then(status => ready.fulfill(status), onError);
-  }
-
-  function onError(e) {
-    // Some servers don't support the status command. If they are able to
-    // response with an error, then can consider the server ready.
-    if (e instanceof error.UnsupportedOperationError) {
-      ready.fulfill();
-      return;
+exports.waitForServer = function(url, timeout, opt_cancelToken) {
+  return new Promise((onResolve, onReject) => {
+    let start = Date.now();
+
+    let done = false;
+    let resolve = (status) => {
+      done = true;
+      onResolve(status);
+    };
+    let reject = (err) => {
+      done = true;
+      onReject(err);
+    };
+
+    if (opt_cancelToken) {
+      opt_cancelToken.then(_ => reject(new promise.CancellationError));
     }
 
-    if (Date.now() - start > timeout) {
-      ready.reject(
-          Error('Timed out waiting for the WebDriver server at ' + url));
-    } else {
-      setTimeout(function() {
-        if (ready.promise.isPending()) {
-          checkServerStatus();
-        }
-      }, 50);
+    checkServerStatus();
+    function checkServerStatus() {
+      return getStatus(url).then(status => resolve(status), onError);
     }
-  }
+
+    function onError(e) {
+      // Some servers don't support the status command. If they are able to
+      // response with an error, then can consider the server ready.
+      if (e instanceof error.UnsupportedOperationError) {
+        resolve({});
+        return;
+      }
+
+      if (Date.now() - start > timeout) {
+        reject(Error('Timed out waiting for the WebDriver server at ' + url));
+      } else {
+        setTimeout(function() {
+          if (!done) {
+            checkServerStatus();
+          }
+        }, 50);
+      }
+    }
+  });
 };
 
 
@@ -101,39 +117,59 @@ exports.waitForServer = function(url, timeout) {
  * timeout expires.
  * @param {string} url The URL to poll.
  * @param {number} timeout How long to wait, in milliseconds.
- * @return {!promise.Promise} A promise that will resolve when the
- *     URL responds with 2xx.
+ * @param {Promise=} opt_cancelToken A promise used as a cancellation signal:
+ *     if resolved before the a 2xx response is received, the wait will be
+ *     terminated early with a {@link promise.CancellationError}.
+ * @return {!Promise} A promise that will resolve when a 2xx is received from
+ *     the given URL, or if the wait is cancelled.
  */
-exports.waitForUrl = function(url, timeout) {
-  var client = new HttpClient(url),
-      request = new HttpRequest('GET', ''),
-      ready = promise.defer(),
-      start = Date.now();
-  testUrl();
-  return ready.promise;
-
-  function testUrl() {
-    client.send(request).then(onResponse, onError);
-  }
-
-  function onError() {
-    if (Date.now() - start > timeout) {
-      ready.reject(Error(
-          'Timed out waiting for the URL to return 2xx: ' + url));
-    } else {
-      setTimeout(function() {
-        if (ready.promise.isPending()) {
-          testUrl();
-        }
-      }, 50);
+exports.waitForUrl = function(url, timeout, opt_cancelToken) {
+  return new Promise((onResolve, onReject) => {
+    let client = new HttpClient(url);
+    let request = new HttpRequest('GET', '');
+    let start = Date.now();
+
+    let done = false;
+    let resolve = () => {
+      done = true;
+      onResolve();
+    };
+    let reject = (err) => {
+      done = true;
+      onReject(err);
+    };
+
+    if (opt_cancelToken) {
+      opt_cancelToken.then(_ => reject(new promise.CancellationError));
+    }
+
+    testUrl();
+
+    function testUrl() {
+      client.send(request).then(onResponse, onError);
+    }
+
+    function onError() {
+      if (Date.now() - start > timeout) {
+        reject(Error('Timed out waiting for the URL to return 2xx: ' + url));
+      } else {
+        setTimeout(function() {
+          if (!done) {
+            testUrl();
+          }
+        }, 50);
+      }
     }
-  }
 
-  function onResponse(response) {
-    if (!ready.promise.isPending()) return;
-    if (response.status > 199 && response.status < 300) {
-      return ready.fulfill();
+    function onResponse(response) {
+      if (done) {
+        return;
+      }
+      if (response.status > 199 && response.status < 300) {
+        resolve();
+        return;
+      }
+      onError();
     }
-    onError();
-  }
+  });
 };

package/ie.js

@@ -403,12 +403,15 @@ function createServiceFromCapabilities(capabilities) {
  */
 class Driver extends webdriver.WebDriver {
   /**
+   * Creates a new session for Microsoft's Internet Explorer.
+   *
    * @param {(capabilities.Capabilities|Options)=} opt_config The configuration
    *     options.
    * @param {promise.ControlFlow=} opt_flow The control flow to use,
    *     or {@code null} to use the currently active flow.
+   * @return {!Driver} A new driver instance.
    */
-  constructor(opt_config, opt_flow) {
+  static createSession(opt_config, opt_flow) {
     var caps = opt_config instanceof Options ?
         opt_config.toCapabilities() :
         (opt_config || capabilities.Capabilities.ie());
@@ -416,16 +419,9 @@ class Driver extends webdriver.WebDriver {
     var service = createServiceFromCapabilities(caps);
     var client = service.start().then(url => new http.HttpClient(url));
     var executor = new http.Executor(client);
-    var driver = webdriver.WebDriver.createSession(executor, caps, opt_flow);
-
-    super(driver.getSession(), executor, driver.controlFlow());
-
-    let boundQuit = this.quit.bind(this);
 
-    /** @override */
-    this.quit = function() {
-      return boundQuit().finally(service.kill.bind(service));
-    };
+    return /** @type {!Driver} */(webdriver.WebDriver.createSession(
+        executor, caps, opt_flow, this, () => service.kill()));
   }
 
   /**