browsertime 15.1.1 → 15.4.0

package/lib/chrome/webdriver/chromium.js

@@ -9,12 +9,14 @@ const path = require('path');
 const parseCpuTrace = require('../parseCpuTrace');
 const har = require('../har');
 const harBuilder = require('../../support/har');
+const util = require('../../support/util');
 const webdriver = require('selenium-webdriver');
 const traceCategoriesParser = require('../traceCategoriesParser');
 const pathToFolder = require('../../support/pathToFolder');
 const ChromeDevtoolsProtocol = require('../chromeDevtoolsProtocol');
 const { Android } = require('../../android');
 const unlink = promisify(fs.unlink);
+const rm = promisify(fs.rm);
 
 class Chromium {
   constructor(storageManager, options) {
@@ -408,7 +410,27 @@ class Chromium {
   /**
    * Before the browser is stopped/closed.
    */
-  async afterBrowserStopped() {}
+  async afterBrowserStopped() {
+    // If we supply a user data dir on desktop, we want to clean up before each start
+    if (
+      this.chrome.args &&
+      this.chrome.cleanUserDataDir &&
+      !this.chrome.android
+    ) {
+      const args = util.toArray(this.chrome.args);
+      for (let arg of args) {
+        if (arg.includes('user-data-dir')) {
+          const userDataDir = arg.split('=')[1];
+          try {
+            await rm(userDataDir, { recursive: true });
+            log.info(`Deleted user data dir: ${userDataDir}`);
+          } catch (e) {
+            log.error(`Could not delete user data dir: ${userDataDir}`, e);
+          }
+        }
+      }
+    }
+  }
 
   async getHARs() {
     if (!this.skipHar) {

package/lib/support/cli.js

@@ -287,6 +287,12 @@ module.exports.parseCommandLine = function parseCommandLine() {
         'Prevent Browsertime from setting its default options for Chrome',
       group: 'chrome'
     })
+    .option('chrome.cleanUserDataDir', {
+      type: 'boolean',
+      describe:
+        'If you use --user-data-dir as an argument to Chrome and want to clean that directory between each iteration you should use --chrome.cleanUserDataDir true.',
+      group: 'chrome'
+    })
     .option('cpu', {
       type: 'boolean',
       describe:
@@ -634,6 +640,11 @@ module.exports.parseCommandLine = function parseCommandLine() {
       type: 'boolean',
       describe: 'Collect Contentful Speed Index when you run --visualMetrics.'
     })
+    .option('visualMetricsPortable', {
+      type: 'boolean',
+      describe:
+        'Use the portable visual-metrics processing script (no ImageMagick dependencies).'
+    })
     .option('scriptInput.visualElements', {
       describe:
         'Include specific elements in visual elements. Give the element a name and select it with document.body.querySelector. Use like this: --scriptInput.visualElements name:domSelector see https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Selectors. Add multiple instances to measure multiple elements. Visual Metrics will use these elements and calculate when they are visible and fully rendered.'
@@ -964,7 +975,7 @@ module.exports.parseCommandLine = function parseCommandLine() {
     })
     .option('block', {
       describe:
-        'Domain to block. Add multiple instances to add multiple domains that will be blocked. If you use Chrome you can also use --blockDomainsExcept (that is more performant). Works for Firefox and Chrome.'
+        'Domain to block or URL or URL pattern to block. If you use Chrome you can also use --blockDomainsExcept (that is more performant). Works in Chrome/Edge. For Firefox you can only block domains.'
     })
     .option('percentiles', {
       type: 'array',

package/lib/support/getViewPort.js

@@ -22,7 +22,12 @@ const sizeMap = {
   'Pixel 2': '412x732',
   'Pixel 2 XL': '412x824',
   iPad: '768x1024',
-  'iPad Pro': '1024x1366'
+  'iPad Pro': '1024x1366',
+  'iPhone XR': '414x896',
+  'iPhone SE': '377x668',
+  'iPhone 12 Pro': '390x844',
+  'Pixel 5': '394x852',
+  'Samsung Galaxy S8+': '412x846'
 };
 
 module.exports = function (options) {

package/lib/video/postprocessing/visualmetrics/visualMetrics.js

@@ -16,9 +16,29 @@ const SCRIPT_PATH = path.join(
   'visualmetrics.py'
 );
 
+const PORTABLE_SCRIPT_PATH = path.join(
+  __dirname,
+  '..',
+  '..',
+  '..',
+  '..',
+  'browsertime',
+  'visualmetrics-portable.py'
+);
+
+function getScript(options) {
+  if (options.visualMetricsPortable) {
+    return PORTABLE_SCRIPT_PATH;
+  }
+  return SCRIPT_PATH;
+}
+
 module.exports = {
-  async checkDependencies() {
-    return execa(process.env.PYTHON || 'python', [SCRIPT_PATH, '--check']);
+  async checkDependencies(options) {
+    return execa(process.env.PYTHON || 'python', [
+      getScript(options),
+      '--check'
+    ]);
   },
   async run(
     videoPath,
@@ -46,7 +66,13 @@ module.exports = {
       '--renderignore',
       5,
       '--json',
-      '--viewport'
+      '--viewport',
+      '--viewportretries',
+      60,
+      '--viewportminheight',
+      100,
+      '--viewportminwidth',
+      100
     ];
 
     if (options.visualMetricsPerceptual) {
@@ -103,7 +129,7 @@ module.exports = {
       scriptArgs.push('-vvv');
     }
 
-    scriptArgs.unshift(SCRIPT_PATH);
+    scriptArgs.unshift(getScript(options));
 
     log.debug('Running visualmetrics.py ' + scriptArgs.join(' '));
     log.info('Get visual metrics from the video');

package/package.json

@@ -1,12 +1,12 @@
 {
   "description": "Browsertime",
-  "version": "15.1.1",
+  "version": "15.4.0",
   "bin": "./bin/browsertime.js",
   "dependencies": {
     "@cypress/xvfb": "1.2.4",
     "@devicefarmer/adbkit": "2.11.3",
-    "@sitespeed.io/chromedriver": "98.0.4758-102b",
-    "@sitespeed.io/edgedriver": "98.0.1108-43",
+    "@sitespeed.io/chromedriver": "100.0.4896-20",
+    "@sitespeed.io/edgedriver": "99.0.1150-25",
     "@sitespeed.io/geckodriver": "0.30.0",
     "@sitespeed.io/throttle": "3.0.0",
     "@sitespeed.io/tracium": "0.3.3",