browsertime 14.17.0 → 14.18.0

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Browsertime changelog (we do [semantic versioning](https://semver.org))
 
+## 14.18.0 - 2022-01-24
+### Added
+* Updated to Edge stable in the Docker container.
+
+### Fixed
+* A more safe way to get dockumentURI for Firefox. See PR [#1699](https://github.com/sitespeedio/browsertime/pull/1699) and bug [#1698](https://github.com/sitespeedio/browsertime/issues/1698).
+
 ## 14.17.0 - 2022-01-23
 ### Added
 * New Select command [#1696](https://github.com/sitespeedio/browsertime/pull/1696):
@@ -13,6 +20,8 @@
 
 * New click by name command `click.byName(name)` [#1697](https://github.com/sitespeedio/browsertime/pull/1697).
 
+### Fixed
+* Remove the top 10 rows of the image to handle progress bars on some mobile browser recordings. Thank you [Gregory Mierzwinski](https://github.com/gmierz) for PR [#1687](https://github.com/sitespeedio/browsertime/pull/1687). 
 
 ##  14.16.0 - 2022-01-14
 ### Added 

package/lib/core/seleniumRunner.js

@@ -222,9 +222,23 @@ class SeleniumRunner {
     // query parameters, order fragments, etc.  We don't want to change url itself 'cuz it can be
     // used for indexing collected data and it is possible that normalization could change a key.
     const normalizedURI = new URL(url).toString();
-    const startURI = new URL(
-      await driver.executeScript('return document.documentURI;')
-    ).toString();
+
+    // See  https://github.com/sitespeedio/browsertime/issues/1698
+    const retries = 5;
+    let documentURI;
+    for (var i = 0; i < retries; i++) {
+      documentURI = await driver.executeScript('return document.documentURI;');
+      if (documentURI != null) {
+        break;
+      }
+      await delay(1000);
+    }
+    let startURI;
+    try {
+      startURI = new URL(documentURI).toString();
+    } catch (e) {
+      log.error(`Failed to get documentURI ${documentURI}`, e);
+    }
     if (this.options.webdriverPageload) {
       const clearOrange = `(function() {
         const orange = document.getElementById('browsertime-orange');

package/package.json

@@ -1,6 +1,6 @@
 {
   "description": "Browsertime",
-  "version": "14.17.0",
+  "version": "14.18.0",
   "bin": "./bin/browsertime.js",
   "dependencies": {
     "@sitespeed.io/throttle": "3.0.0",