chrome-devtools-frontend 1.0.937775 → 1.0.938150

package/front_end/third_party/puppeteer/package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "puppeteer",
-  "version": "10.4.0",
+  "version": "11.0.0",
   "description": "A high-level API to control headless Chrome over the DevTools Protocol",
   "main": "./cjs-entry.js",
   "types": "lib/types.d.ts",
@@ -17,7 +17,7 @@
     "assert-unit-coverage": "cross-env COVERAGE=1 mocha --config mocha-config/coverage-tests.js",
     "funit": "cross-env PUPPETEER_PRODUCT=firefox npm run unit",
     "test": "npm run tsc && npm run lint --silent && npm run unit-with-coverage && npm run test-browser",
-    "prepare": "node typescript-if-required.js",
+    "prepare": "node typescript-if-required.js && husky install",
     "prepublishOnly": "npm run build",
     "dev-install": "npm run tsc && node install.js",
     "install": "node install.js",
@@ -58,67 +58,60 @@
   "author": "The Chromium Authors",
   "license": "Apache-2.0",
   "dependencies": {
-    "debug": "4.3.1",
+    "debug": "4.3.2",
     "devtools-protocol": "0.0.901419",
     "extract-zip": "2.0.1",
     "https-proxy-agent": "5.0.0",
-    "node-fetch": "2.6.1",
+    "node-fetch": "2.6.5",
     "pkg-dir": "4.2.0",
-    "progress": "2.0.1",
+    "progress": "2.0.3",
     "proxy-from-env": "1.1.0",
     "rimraf": "3.0.2",
-    "tar-fs": "2.0.0",
-    "unbzip2-stream": "1.3.3",
-    "ws": "7.4.6"
+    "tar-fs": "2.1.1",
+    "unbzip2-stream": "1.4.3",
+    "ws": "8.2.3"
   },
   "devDependencies": {
-    "@commitlint/cli": "11.0.0",
-    "@commitlint/config-conventional": "11.0.0",
-    "@microsoft/api-documenter": "7.13.8",
-    "@microsoft/api-extractor": "7.15.1",
-    "@types/debug": "0.0.31",
+    "@commitlint/cli": "13.2.0",
+    "@commitlint/config-conventional": "13.2.0",
+    "@microsoft/api-documenter": "7.13.65",
+    "@microsoft/api-extractor": "7.18.15",
+    "@types/debug": "4.1.7",
     "@types/mime": "2.0.3",
-    "@types/mocha": "7.0.2",
-    "@types/node": "14.14.45",
+    "@types/mocha": "9.0.0",
+    "@types/node": "16.10.9",
     "@types/proxy-from-env": "1.0.1",
-    "@types/rimraf": "2.0.2",
-    "@types/sinon": "9.0.11",
-    "@types/tar-fs": "1.16.2",
-    "@types/ws": "7.4.4",
+    "@types/rimraf": "3.0.2",
+    "@types/sinon": "10.0.4",
+    "@types/tar-fs": "2.0.1",
+    "@types/ws": "8.2.0",
     "@typescript-eslint/eslint-plugin": "4.23.0",
-    "@typescript-eslint/parser": "4.23.0",
-    "@web/test-runner": "0.12.20",
+    "@typescript-eslint/parser": "4.33.0",
+    "@web/test-runner": "0.13.18",
     "commonmark": "0.29.3",
     "cross-env": "7.0.3",
-    "eslint": "7.26.0",
+    "eslint": "7.32.0",
     "eslint-config-prettier": "8.3.0",
     "eslint-plugin-import": "2.22.1",
-    "eslint-plugin-mocha": "8.1.0",
-    "eslint-plugin-prettier": "3.4.0",
-    "eslint-plugin-unicorn": "22.0.0",
-    "esprima": "4.0.0",
+    "eslint-plugin-mocha": "9.0.0",
+    "eslint-plugin-prettier": "4.0.0",
+    "eslint-plugin-unicorn": "37.0.1",
+    "esprima": "4.0.1",
     "expect": "25.2.7",
-    "husky": "4.3.8",
+    "husky": "7.0.2",
     "jpeg-js": "0.3.7",
     "mime": "2.5.2",
     "minimist": "1.2.0",
-    "mocha": "8.4.0",
+    "mocha": "9.1.3",
     "ncp": "2.0.0",
-    "pixelmatch": "4.0.2",
-    "pngjs": "5.0.0",
+    "pixelmatch": "5.2.1",
+    "pngjs": "6.0.0",
     "prettier": "2.3.0",
     "sinon": "9.2.4",
     "source-map-support": "0.5.19",
-    "standard-version": "9.3.0",
+    "standard-version": "9.3.2",
     "text-diff": "1.0.1",
-    "ts-node": "9.1.1",
-    "typescript": "4.2.4"
-  },
-  "husky": {
-    "hooks": {
-      "pre-commit": "npm run eslint",
-      "commit-msg": "commitlint --env HUSKY_GIT_PARAMS",
-      "pre-push": "npm run tsc && npm run eslint && npm run doc && npm run ensure-pinned-deps"
-    }
+    "ts-node": "10.4.0",
+    "typescript": "4.4.4"
   }
 }

package/front_end/ui/components/icon_button/iconButton.css

@@ -61,6 +61,7 @@
     background-color: ButtonFace;
   }
 
+  .icon-button:focus-visible,
   .icon-button.with-click-handler:hover {
     background-color: Highlight;
     color: HighlightText;

package/front_end/ui/legacy/XLink.ts

@@ -15,9 +15,6 @@ import {addReferrerToURLIfNecessary, copyLinkAddressLabel, MaxLengthForDisplayed
 import {XElement} from './XElement.js';
 
 export class XLink extends XElement {
-  tabIndex: number;
-  target: string;
-  rel: string;
   hrefInternal: string|null;
   private clickable: boolean;
   private readonly onClick: (arg0: Event) => void;
@@ -41,9 +38,9 @@ export class XLink extends XElement {
 
     this.style.setProperty('display', 'inline');
     ARIAUtils.markAsLink(this);
-    this.tabIndex = 0;
-    this.target = '_blank';
-    this.rel = 'noopener';
+    this.setAttribute('tabindex', '0');
+    this.setAttribute('target', '_blank');
+    this.setAttribute('rel', 'noopener');
 
     this.hrefInternal = null;
     this.clickable = true;

package/front_end/ui/legacy/inspectorSyntaxHighlight.css

@@ -113,7 +113,7 @@
 
 .cm-css-property,
 .webkit-css-property {
-  color: rgb(200 0 0); /* stylelint-disable-line plugin/use_theme_colors */ /* See: crbug.com/1152736 for color variable migration. */
+  color: var(--webkit-css-property-color, rgb(200 0 0)); /* stylelint-disable-line plugin/use_theme_colors */ /* See: crbug.com/1152736 for color variable migration. */
 }
 
 .cm-xml-meta {

package/front_end/ui/legacy/inspectorSyntaxHighlightDark.css

@@ -213,7 +213,7 @@
 }
 
 .webkit-css-property {
-  color: rgb(53 212 199); /* stylelint-disable-line plugin/use_theme_colors */ /* See: crbug.com/1152736 for color variable migration. */
+  color: var(--webkit-css-property-color, rgb(53 212 199)); /* stylelint-disable-line plugin/use_theme_colors */ /* See: crbug.com/1152736 for color variable migration. */
 }
 
 .cm-def {

package/front_end/ui/legacy/tabbedPane.css

@@ -390,6 +390,11 @@
     color: HighlightText;
   }
 
+  .tabbed-pane-header-tab:hover .tabbed-pane-close-button,
+  .tabbed-pane-shadow .tabbed-pane-header-tab:focus-visible .tabbed-pane-close-button {
+    --icon-color: HighlightText;
+  }
+
   .tabbed-pane-header-tabs-drop-down-container {
     opacity: 100%;
   }

package/front_end/ui/legacy/treeoutline.css

@@ -215,4 +215,20 @@ ol.tree-outline:not(.hide-selection-when-blurred) li.selected:focus * {
     forced-color-adjust: none;
     color: HighlightText;
   }
+
+  .tree-outline:not(.hide-selection-when-blurred) li.selected:focus-visible devtools-adorner,
+  .tree-outline.hide-selection-when-blurred li.selected:focus-visible devtools-adorner {
+    --override-adorner-background-color: Highlight;
+    --override-adorner-border-color: HighlightText;
+  }
+
+  .tree-outline:not(.hide-selection-when-blurred) devtools-icon,
+  .tree-outline.hide-selection-when-blurred devtools-icon {
+    --icon-color: ButtonText;
+  }
+
+  .tree-outline:not(.hide-selection-when-blurred) .selected devtools-icon,
+  .tree-outline.hide-selection-when-blurred .selected:focus-visible devtools-icon {
+    --icon-color: HighlightText;
+  }
 }

package/front_end/ui/legacy/viewContainers.css

@@ -71,7 +71,12 @@
 @media (forced-colors: active) {
   .expandable-view-title:focus-visible {
     forced-color-adjust: none;
-    background: canvas !important; /* stylelint-disable-line declaration-no-important */
+    color: HighlightText;
+    background-color: Highlight;
     box-shadow: 0 0 0 2px Highlight inset;
   }
+
+  .expandable-view-title:focus-visible .title-expand-icon {
+    background-color: HighlightText;
+  }
 }

package/package.json

@@ -55,5 +55,5 @@
     "unittest": "scripts/test/run_unittests.py --no-text-coverage",
     "watch": "third_party/node/node.py --output scripts/watch_build.js"
   },
-  "version": "1.0.937775"
+  "version": "1.0.938150"
 }

package/scripts/eslint_rules/lib/check_test_definitions.js

@@ -47,7 +47,8 @@ module.exports = {
   create: function(context) {
     return {
       MemberExpression(node) {
-        if (node.object.name === 'it' && (node.property.name === 'skip' || node.property.name === 'skipOnPlatforms') &&
+        if ((node.object.name === 'it' || node.object.name === 'describe') &&
+            (node.property.name === 'skip' || node.property.name === 'skipOnPlatforms') &&
             node.parent.type === 'CallExpression') {
           const testNameNode = node.property.name === 'skip' ? node.parent.arguments[0] : node.parent.arguments[1];
 

package/scripts/eslint_rules/tests/check_test_definitions_test.js

@@ -62,6 +62,24 @@ ruleTester.run('check_e2e_tests', rule, {
       `,
       filename: 'test/e2e/folder/file.ts',
     },
+    {
+      code: `import {describe, it} from '../../shared/mocha-extensions.js';
+      // Explaining comment
+      describe.skip('[crbug.com/123456]: e2e-test', async () => {
+      });
+      `,
+      filename: 'test/e2e/folder/file.ts',
+      errors: [{message: rule.meta.messages.missingBugId}],
+    },
+    {
+      code: `import {describe, it} from '../../shared/mocha-extensions.js';
+      // Explaining comment
+      describe.skipOnPlatforms(['mac'], '[crbug.com/123456]: e2e-test', async () => {
+      });
+      `,
+      filename: 'test/e2e/folder/file.ts',
+      errors: [{message: rule.meta.messages.missingBugId}],
+    },
   ],
 
   invalid: [
@@ -111,5 +129,21 @@ ruleTester.run('check_e2e_tests', rule, {
       filename: 'test/e2e/folder/file.ts',
       errors: [{message: rule.meta.messages.extraBugId}],
     },
+    {
+      code: `import {describe, it} from '../../shared/mocha-extensions.js';
+      describe.skip('e2e-test', async () => {
+      });
+      `,
+      filename: 'test/e2e/folder/file.ts',
+      errors: [{message: rule.meta.messages.missingBugId}, {message: rule.meta.messages.comment}],
+    },
+    {
+      code: `import {describe, it} from '../../shared/mocha-extensions.js';
+      describe.skipOnPlatforms(['mac'], 'e2e-test', async () => {
+      });
+      `,
+      filename: 'test/e2e/folder/file.ts',
+      errors: [{message: rule.meta.messages.missingBugId}, {message: rule.meta.messages.comment}],
+    },
   ]
 });

package/scripts/eslint_rules/tests/migrate_create_shadow_root_with_styles_test.js

@@ -12,7 +12,7 @@ const ruleTester = new (require('eslint').RuleTester)({
 
 const MIGRATION_ERROR_MESSAGE = 'Import CSS file instead of passing a string into createShadowRootWithStyles';
 const MANUALLY_MIGRATE_ERROR_ESSAGE =
-    'Please manually migrate this file. Got error: Cannot read property \'properties\' of undefined';
+    'Please manually migrate this file. Got error: Cannot read properties of undefined (reading \'properties\')';
 
 ruleTester.run('check_migrate_RegisterRequiredCSS', rule, {
   valid: [

package/scripts/eslint_rules/tests/migrate_register_required_css_test.js

@@ -15,7 +15,7 @@ const ADD_WAS_SHOW_EXPECTED_ERROR_MESSAGE =
 const EDIT_WAS_SHOW_EXPECTED_ERROR_MESSAGE =
     'Import CSS file instead of using registerRequiredCSS and edit wasShown method';
 const MANUALLY_MIGRATE_ERROR_ESSAGE =
-    'Please manually migrate components/test.css as it has edge cases not covered by this script. Got error: Cannot read property \'range\' of undefined.';
+    'Please manually migrate components/test.css as it has edge cases not covered by this script. Got error: Cannot read properties of undefined (reading \'range\').';
 
 ruleTester.run('check_migrate_RegisterRequiredCSS', rule, {
   valid: [

package/front_end/panels/elements/module.json

@@ -1,5 +0,0 @@
-{
-  "dependencies": [
-    "ui/legacy"
-  ]
-}

package/front_end/panels/protocol_monitor/module.json

@@ -1,5 +0,0 @@
-{
-  "dependencies": [
-    "ui/legacy"
-  ]
-}

package/front_end/panels/sources/module.json

@@ -1,5 +0,0 @@
-{
-  "dependencies": [
-    "ui/legacy"
-  ]
-}