browsertime 16.12.0 → 16.13.2

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Browsertime changelog (we do [semantic versioning](https://semver.org))
 
+
+## 16.13.2 - 2022-08-14
+### Fixed
+* Disable adding lastCPULongTask to the HAR data since it breaks waterfall view on slow mobile phones with late CPU long tasks [#1828](https://github.com/sitespeedio/browsertime/pull/1828).
+
+## 16.13.1 - 2022-08-10
+### Fixed
+* Make sure that Android id is always picked up from the phone if tests run on Android [#1826](https://github.com/sitespeedio/browsertime/pull/1826).
+
+## 16.13.0 - 2022-08-08
+### Added
+* Upgraded to Edgedriver 104.
+
+### Fixed
+* Upgraded to Selenium 4.3.1 [#1824](https://github.com/sitespeedio/browsertime/pull/1824).
+* Upgraded Chrome reomte interface to 0.31.3 [#1825](https://github.com/sitespeedio/browsertime/pull/1825)
 ## 16.12.0 - 2022-08-06
 ### Added
 * Upgraded to Chrome 104, Edge 104, Firefox 103 in the Docker container.

package/lib/android/index.js

@@ -161,13 +161,16 @@ class Android {
     const name = (await adb.util.readAll(rawName)).toString().trim();
     const rawDevice = await this._runCommand(`getprop ro.product.device`);
     const device = (await adb.util.readAll(rawDevice)).toString().trim();
+    const rawId = await this._runCommand(`getprop ro.serialno`);
+    const id = (await adb.util.readAll(rawId)).toString().trim();
+
     const rawRelease = await this._runCommand(
       `getprop ro.build.version.release `
     );
     const androidVersion = (await adb.util.readAll(rawRelease))
       .toString()
       .trim();
-    return { model, name, device, androidVersion };
+    return { model, name, device, androidVersion, id };
   }
 
   async pullNetLog(destination) {

package/lib/chrome/webdriver/builder.js

@@ -8,7 +8,6 @@ const isEmpty = require('lodash.isempty');
 const get = require('lodash.get');
 const setupChromiumOptions = require('./setupChromiumOptions');
 
-let hasConfiguredChromeDriverService = false;
 /**
  * Configure a WebDriver builder based on the specified options.
  * @param builder
@@ -17,34 +16,31 @@ let hasConfiguredChromeDriverService = false;
 module.exports.configureBuilder = function (builder, baseDir, options) {
   const chromeConfig = options.chrome || {};
 
-  if (!hasConfiguredChromeDriverService) {
-    let chromedriverPath = get(chromeConfig, 'chromedriverPath');
-    if (!chromedriverPath) {
-      const chromedriver = require('@sitespeed.io/chromedriver');
-      chromedriverPath = chromedriver.binPath();
-    }
-
-    const serviceBuilder = new chrome.ServiceBuilder(chromedriverPath);
-
-    // Remove the check that matches the Chromedriver version with Chrome version.
-    serviceBuilder.addArguments('--disable-build-check');
-
-    if (options.chrome && options.chrome.chromedriverPort) {
-      serviceBuilder.setPort(options.chrome.chromedriverPort);
-    }
-    if (
-      options.verbose >= 2 ||
-      chromeConfig.enableChromeDriverLog ||
-      chromeConfig.enableVerboseChromeDriverLog
-    ) {
-      serviceBuilder.loggingTo(`${baseDir}/chromedriver.log`);
-      if (options.verbose >= 3 || chromeConfig.enableVerboseChromeDriverLog)
-        serviceBuilder.enableVerboseLogging();
-    }
-    chrome.setDefaultService(serviceBuilder.build());
-    hasConfiguredChromeDriverService = true;
+  let chromedriverPath = get(chromeConfig, 'chromedriverPath');
+  if (!chromedriverPath) {
+    const chromedriver = require('@sitespeed.io/chromedriver');
+    chromedriverPath = chromedriver.binPath();
   }
 
+  const serviceBuilder = new chrome.ServiceBuilder(chromedriverPath);
+
+  // Remove the check that matches the Chromedriver version with Chrome version.
+  serviceBuilder.addArguments('--disable-build-check');
+
+  if (options.chrome && options.chrome.chromedriverPort) {
+    serviceBuilder.setPort(options.chrome.chromedriverPort);
+  }
+  if (
+    options.verbose >= 2 ||
+    chromeConfig.enableChromeDriverLog ||
+    chromeConfig.enableVerboseChromeDriverLog
+  ) {
+    serviceBuilder.loggingTo(`${baseDir}/chromedriver.log`);
+    if (options.verbose >= 3 || chromeConfig.enableVerboseChromeDriverLog)
+      serviceBuilder.enableVerboseLogging();
+  }
+  builder.setChromeService(serviceBuilder);
+
   const proxyPacSettings = pick(options.proxy, ['pac']);
 
   if (!isEmpty(proxyPacSettings)) {

package/lib/support/har/index.js

@@ -91,9 +91,11 @@ function addExtrasToHAR(
     harPageTimings._firstPaint = timings.firstPaint;
   }
 
+  /*
   if (cpu && cpu.longTasks && cpu.longTasks.lastLongTask) {
     harPageTimings._lastCPULongTask = cpu.longTasks.lastLongTask;
   }
+  */
 
   if (timings && timings.largestContentfulPaint) {
     harPageTimings._largestContentfulPaint =

package/package.json

@@ -1,18 +1,18 @@
 {
   "description": "Browsertime",
-  "version": "16.12.0",
+  "version": "16.13.2",
   "bin": "./bin/browsertime.js",
   "dependencies": {
     "@cypress/xvfb": "1.2.4",
     "@devicefarmer/adbkit": "2.11.3",
     "@sitespeed.io/chromedriver": "104.0.5112-29",
-    "@sitespeed.io/edgedriver": "103.0.1264-37",
+    "@sitespeed.io/edgedriver": "104.0.1293-47",
     "@sitespeed.io/geckodriver": "0.31.0-c",
     "@sitespeed.io/throttle": "4.0.1",
     "@sitespeed.io/tracium": "0.3.3",
     "btoa": "1.2.1",
     "chrome-har": "0.13.0",
-    "chrome-remote-interface": "0.31.2",
+    "chrome-remote-interface": "0.31.3",
     "dayjs": "1.11.1",
     "execa": "5.1.1",
     "fast-stats": "0.0.6",
@@ -26,7 +26,7 @@
     "lodash.merge": "4.6.2",
     "lodash.pick": "4.4.0",
     "lodash.set": "4.3.2",
-    "selenium-webdriver": "4.3.0",
+    "selenium-webdriver": "4.3.1",
     "yargs": "17.4.1"
   },
   "optionalDependencies": {