chrome-devtools-frontend 1.0.964440 → 1.0.966086

package/front_end/ui/components/docs/tree_outline/custom-renderers.ts

@@ -38,13 +38,13 @@ const data: TreeOutline.TreeOutline.TreeOutlineData<TreeNodeData> = {
     {
       treeNodeData: {cssProperty: 'margin', cssValue: '10px 5px'},
       id: '3',
-      async children() {
-        return Promise.resolve<TreeOutline.TreeOutlineUtils.TreeNode<TreeNodeData>[]>([
+      async children(): Promise<TreeOutline.TreeOutlineUtils.TreeNode<TreeNodeData>[]> {
+        return [
           {treeNodeData: {cssProperty: 'margin-left', cssValue: '5px'}, id: '4'},
           {treeNodeData: {cssProperty: 'margin-right', cssValue: '5px'}, id: '5'},
           {treeNodeData: {cssProperty: 'margin-top', cssValue: '10px'}, id: '6'},
           {treeNodeData: {cssProperty: 'margin-bottom', cssValue: '10px'}, id: '7'},
-        ]);
+        ];
       },
     },
   ],

package/front_end/ui/legacy/Icon.ts

@@ -131,8 +131,8 @@ const initialDescriptors = new Map<string, Descriptor>([
   ['smallicon-warning', {position: 'd1', spritesheet: 'smallicons'}],
   ['smallicon-network-product', {position: 'e1', spritesheet: 'smallicons'}],
   ['smallicon-clear-warning', {position: 'f1', spritesheet: 'smallicons', isMask: true}],
-  ['smallicon-clear-info', {position: 'f2', spritesheet: 'smallicons'}],
-  ['smallicon-clear-error', {position: 'f3', spritesheet: 'smallicons'}],
+  ['smallicon-clear-info', {position: 'f2', spritesheet: 'smallicons', isMask: true}],
+  ['smallicon-clear-error', {position: 'f3', spritesheet: 'smallicons', isMask: true}],
   ['smallicon-account-circle', {position: 'f4', spritesheet: 'smallicons'}],
   ['smallicon-videoplayer-paused', {position: 'f6', spritesheet: 'smallicons', isMask: true}],
   ['smallicon-videoplayer-playing', {position: 'g6', spritesheet: 'smallicons', isMask: true}],

package/front_end/ui/legacy/components/object_ui/ObjectPropertiesSection.ts

@@ -1612,12 +1612,12 @@ export class Renderer implements UI.UIUtils.Renderer {
     return rendererInstance;
   }
 
-  render(object: Object, options?: UI.UIUtils.Options): Promise<{
+  async render(object: Object, options?: UI.UIUtils.Options): Promise<{
     node: Node,
     tree: UI.TreeOutline.TreeOutline|null,
   }|null> {
     if (!(object instanceof SDK.RemoteObject.RemoteObject)) {
-      return Promise.reject(new Error('Can\'t render ' + object));
+      throw new Error('Can\'t render ' + object);
     }
     options = options || {title: undefined, editable: undefined};
     const title = options.title;
@@ -1626,10 +1626,10 @@ export class Renderer implements UI.UIUtils.Renderer {
       section.titleLessMode();
     }
     section.editable = Boolean(options.editable);
-    return Promise.resolve(({node: section.element, tree: section} as {
+    return {node: section.element, tree: section} as {
       node: Node,
       tree: UI.TreeOutline.TreeOutline | null,
-    } | null));
+    } | null;
   }
 }
 

package/front_end/ui/legacy/components/perf_ui/LiveHeapProfile.ts

@@ -36,8 +36,8 @@ export class LiveHeapProfile implements Common.Runnable.Runnable,
     return liveHeapProfileInstance;
   }
 
-  run(): Promise<void> {
-    return Promise.resolve();
+  async run(): Promise<void> {
+    return;
   }
 
   modelAdded(model: SDK.HeapProfilerModel.HeapProfilerModel): void {

package/front_end/ui/legacy/themeColors.css

@@ -267,7 +267,7 @@
   --color-input-text-disabled: rgba(128 134 139 / 70%);
   /* Colors for styling buttons */
   --color-button-outline-focus: rgb(138 180 248 / 75%);
-  --color-button-primary-background-hovering: rgb(174 203 250 / 100%);
+  --color-button-primary-background-hovering: rgb(128 167 229 / 100%);
   --color-button-primary-background-pressed: rgb(210 227 252 / 100%);
   --color-button-primary-text: rgb(0 0 0);
   --color-button-secondary-background-hovering: rgb(138 180 248 / 15%);

package/package.json

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

package/scripts/build/devtools_plugin.js

@@ -42,6 +42,16 @@ function devtoolsPlugin(source, importer) {
   if (!importer) {
     return null;
   }
+
+  if (source === '../../lib/codemirror' || source === 'fs') {
+    // These are imported via require(...), but we don't use
+    // @rollup/plugin-commonjs. So this check is not necessary for rollup. But
+    // need to have this for esbuild as it doesn't ignore require(...).
+    return {
+      external: true,
+    };
+  }
+
   const currentDirectory = path.normalize(dirnameWithSeparator(importer));
   const importedFilelocation = path.normalize(path.join(currentDirectory, source));
   const importedFileDirectory = dirnameWithSeparator(importedFilelocation);

package/scripts/build/esbuild.js

@@ -0,0 +1,62 @@
+// Copyright 2022 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// @ts-check
+
+const path = require('path');
+
+const devtools_paths = require('../devtools_paths.js');
+const devtools_plugin = require('./devtools_plugin.js');
+
+// esbuild module uses binary in this path.
+process.env.ESBUILD_BINARY_PATH = path.join(devtools_paths.devtoolsRootPath(), 'third_party', 'esbuild', 'esbuild');
+
+const entryPoints = [process.argv[2]];
+const outfile = process.argv[3];
+
+const outdir = path.dirname(outfile);
+
+const plugin = {
+  name: 'devtools-plugin',
+  setup(build) {
+    // https://esbuild.github.io/plugins/#on-resolve
+    build.onResolve({filter: /.*/}, args => {
+      const res = devtools_plugin.devtoolsPlugin(args.path, args.importer);
+      if (!res) {
+        return null;
+      }
+
+      if (res.external && res.id) {
+        return {
+          external: res.external,
+          path: './' + path.relative(outdir, res.id),
+        };
+      }
+
+      if (res.external) {
+        return {
+          external: true,
+        };
+      }
+
+      return {
+        path: res.id,
+      };
+    });
+  },
+};
+
+require('esbuild')
+    .build({
+      entryPoints,
+      outfile,
+      bundle: true,
+      format: 'esm',
+      platform: 'browser',
+      plugins: [plugin],
+    })
+    .catch(err => {
+      console.error('failed to run esbuild:', err);
+      process.exit(1);
+    });

package/scripts/build/ninja/bundle.gni

@@ -2,46 +2,91 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import("../../../third_party/typescript/typescript.gni")
 import("./node.gni")
 import("./vars.gni")
 
+declare_args() {
+  # If this is enabled, devtools build uses esbuild instead of rollup.js to
+  # bundle JavaScript files.
+  devtools_fast_bundle = devtools_skip_typecheck
+}
+
+assert(!(devtools_fast_bundle && is_official_build),
+       "Official build should not bundle with esbuild")
+
 template("bundle") {
   assert(defined(invoker.entrypoint),
          "You must define the 'entrypoint' for a bundle target")
 
-  node_action(target_name) {
-    script = "node_modules/rollup/dist/bin/rollup"
-
-    forward_variables_from(invoker,
-                           [
-                             "visibility",
-                             "deps",
-                             "public_deps",
-                           ])
-
-    inputs = [
-      invoker.entrypoint,
-      devtools_location_prepend + "scripts/build/rollup.config.js",
-    ]
-
-    args = [
-      # TODO(crbug.com/1098074): We need to hide warnings that are written stderr,
-      # as Chromium does not process the returncode of the subprocess correctly
-      # and instead looks if `stderr` is empty.
-      "--silent",
-      "--config",
-      rebase_path(devtools_location_prepend + "scripts/build/rollup.config.js",
-                  root_build_dir),
-      "--input",
-      rebase_path(invoker.entrypoint, root_build_dir),
-      "--file",
-      rebase_path(invoker.output_file_location, root_build_dir),
-    ]
-
-    if (!devtools_dcheck_always_on) {
-      args += [ "--configDCHECK" ]
+  if (devtools_fast_bundle) {
+    node_action(target_name) {
+      script = "scripts/build/esbuild.js"
+      forward_variables_from(invoker,
+                             [
+                               "visibility",
+                               "deps",
+                               "public_deps",
+                             ])
+
+      inputs = [
+        invoker.entrypoint,
+        devtools_location_prepend + "scripts/build/devtools_plugin.js",
+        devtools_location_prepend + "scripts/devtools_paths.js",
+      ]
+
+      _esbuild = devtools_location_prepend + "third_party/esbuild/esbuild"
+      if (host_os == "win") {
+        inputs += [ _esbuild + ".exe" ]
+      } else {
+        inputs += [ _esbuild ]
+      }
+
+      args = [
+        rebase_path(invoker.entrypoint, root_build_dir),
+        rebase_path(invoker.output_file_location, root_build_dir),
+      ]
+
+      outputs = [ invoker.output_file_location ]
     }
+  } else {
+    node_action(target_name) {
+      script = "node_modules/rollup/dist/bin/rollup"
+
+      forward_variables_from(invoker,
+                             [
+                               "visibility",
+                               "deps",
+                               "public_deps",
+                             ])
+
+      inputs = [
+        invoker.entrypoint,
+        devtools_location_prepend + "scripts/build/rollup.config.js",
+        devtools_location_prepend + "scripts/build/devtools_plugin.js",
+        devtools_location_prepend + "scripts/devtools_paths.js",
+      ]
 
-    outputs = [ invoker.output_file_location ]
+      args = [
+        # TODO(crbug.com/1098074): We need to hide warnings that are written stderr,
+        # as Chromium does not process the returncode of the subprocess correctly
+        # and instead looks if `stderr` is empty.
+        "--silent",
+        "--config",
+        rebase_path(
+            devtools_location_prepend + "scripts/build/rollup.config.js",
+            root_build_dir),
+        "--input",
+        rebase_path(invoker.entrypoint, root_build_dir),
+        "--file",
+        rebase_path(invoker.output_file_location, root_build_dir),
+      ]
+
+      if (!devtools_dcheck_always_on) {
+        args += [ "--configDCHECK" ]
+      }
+
+      outputs = [ invoker.output_file_location ]
+    }
   }
 }