chrome-devtools-frontend 1.0.1473514 → 1.0.1506453

package/front_end/global_typings/global_defs.d.ts

@@ -56,6 +56,11 @@ interface Scheduler {
 
 interface Window {
   readonly scheduler: Scheduler;
+
+  // Chromium only feature so not exposed on TypeScript lib.dom
+  showSaveFilePicker(opts: {
+    suggestedName: string,
+  }): Promise<FileSystemFileHandle>;
 }
 
 interface WorkerGlobalScope {

package/front_end/legacy_test_runner/accessibility_test_runner/accessibility_test_runner.js

@@ -9,7 +9,7 @@ import * as Accessibility from '../../panels/accessibility/accessibility.js';
 import {TestRunner} from '../test_runner/test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const AccessibilityTestRunner = {};

package/front_end/legacy_test_runner/application_test_runner/CacheStorageTestRunner.js

@@ -7,7 +7,7 @@ import * as TextUtils from '../../models/text_utils/text_utils.js';
 import * as Application from '../../panels/application/application.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const dumpCacheTree = async function(pathFilter) {

package/front_end/legacy_test_runner/application_test_runner/IndexedDBTestRunner.js

@@ -7,7 +7,7 @@ import * as Application from '../../panels/application/application.js';
 import {ConsoleTestRunner} from '../console_test_runner/console_test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const dumpIndexedDBTree = async function() {

package/front_end/legacy_test_runner/application_test_runner/ResourceTreeTestRunner.js

@@ -9,7 +9,7 @@ import * as UI from '../../ui/legacy/legacy.js';
 import {SourcesTestRunner} from '../sources_test_runner/sources_test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const dumpResources = function(formatter) {

package/front_end/legacy_test_runner/application_test_runner/ResourcesTestRunner.js

@@ -7,7 +7,7 @@ import * as SDK from '../../core/sdk/sdk.js';
 import * as Application from '../../panels/application/application.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 /**

package/front_end/legacy_test_runner/application_test_runner/ServiceWorkersTestRunner.js

@@ -6,7 +6,7 @@ import * as SDK from '../../core/sdk/sdk.js';
 import * as Application from '../../panels/application/application.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const registerServiceWorker = function(script, scope) {

package/front_end/legacy_test_runner/application_test_runner/StorageTestRunner.js

@@ -5,7 +5,7 @@
 import * as Application from '../../panels/application/application.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const isStorageView = function(view) {

package/front_end/legacy_test_runner/axe_core_test_runner/axe_core_test_runner.js

@@ -7,7 +7,7 @@ import '../../third_party/axe-core/axe.js';
 import {TestRunner} from '../test_runner/test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 // These rules are disabled for one or more of the following reasons:

package/front_end/legacy_test_runner/bindings_test_runner/AutomappingTestRunner.js

@@ -9,7 +9,7 @@ import * as TextUtils from '../../models/text_utils/text_utils.js';
 import * as Workspace from '../../models/workspace/workspace.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const addFiles = function(testFileSystem, files) {

package/front_end/legacy_test_runner/bindings_test_runner/BindingsTestRunner.js

@@ -3,7 +3,7 @@
 // found in the LICENSE file.
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 import * as Platform from '../../core/platform/platform.js';

package/front_end/legacy_test_runner/bindings_test_runner/IsolatedFilesystemTestRunner.js

@@ -7,7 +7,7 @@ import * as Host from '../../core/host/host.js';
 import * as Persistence from '../../models/persistence/persistence.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 Host.InspectorFrontendHost.InspectorFrontendHostInstance.isolatedFileSystem = function(name) {

package/front_end/legacy_test_runner/bindings_test_runner/OverridesTestRunner.js

@@ -10,7 +10,7 @@ import {TestFileSystem} from './IsolatedFilesystemTestRunner.js';
 
 /**
  * @param {string} folderPath
- * @return {!Promise<!{isolatedFileSystem: !Persistence.IsolatedFileSystem.IsolatedFileSystem, project: !Workspace.Workspace.Project, testFileSystem: !BindingsTestRunner.TestFileSystem}>}
+ * @returns {!Promise<!{isolatedFileSystem: !Persistence.IsolatedFileSystem.IsolatedFileSystem, project: !Workspace.Workspace.Project, testFileSystem: !BindingsTestRunner.TestFileSystem}>}
  */
 export const createOverrideProject = async function(folderPath) {
   const testFileSystem = new TestFileSystem(folderPath);

package/front_end/legacy_test_runner/bindings_test_runner/PersistenceTestRunner.js

@@ -6,7 +6,7 @@ import * as Persistence from '../../models/persistence/persistence.js';
 import * as Workspace from '../../models/workspace/workspace.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 Persistence.Persistence.PersistenceBinding.prototype.toString = function() {

package/front_end/legacy_test_runner/console_test_runner/console_test_runner.js

@@ -11,7 +11,7 @@ import * as UI from '../../ui/legacy/legacy.js';
 import {TestRunner} from '../test_runner/test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 export const ConsoleTestRunner = {};
@@ -33,7 +33,7 @@ ConsoleTestRunner.dumpConsoleMessages = async function(printOriginatingCommand,
  * @param {boolean=} printOriginatingCommand
  * @param {boolean=} dumpClassNames
  * @param {!ConsoleTestRunner.Formatter=} formatter
- * @return {!Promise<!Array<string>>}
+ * @returns {!Promise<!Array<string>>}
  */
 ConsoleTestRunner.dumpConsoleMessagesIntoArray = async function(printOriginatingCommand, dumpClassNames, formatter) {
   formatter = formatter || ConsoleTestRunner.prepareConsoleMessageText;
@@ -98,7 +98,7 @@ ConsoleTestRunner.dumpConsoleMessagesIntoArray = async function(printOriginating
 
 /**
  * @param {!Element} messageElement
- * @return {string}
+ * @returns {string}
  */
 ConsoleTestRunner.prepareConsoleMessageText = function(messageElement) {
   let messageText = messageElement.deepTextContent().replace(/\u200b/g, '');
@@ -118,7 +118,7 @@ ConsoleTestRunner.prepareConsoleMessageText = function(messageElement) {
 
 /**
  * @param {!Element} messageElement
- * @return {string}
+ * @returns {string}
  */
 ConsoleTestRunner.prepareConsoleMessageTextTrimmed = function(messageElement) {
   return ConsoleTestRunner.prepareConsoleMessageText(messageElement).replace(/[ ]+/g, ' ');
@@ -128,7 +128,7 @@ ConsoleTestRunner.prepareConsoleMessageTextTrimmed = function(messageElement) {
  * @param {!Console.ConsoleViewMessage.ConsoleViewMessage} viewMessage
  * @param {boolean} forceInvalidate
  * @param {!Array<string>} results
- * @return {boolean}
+ * @returns {boolean}
  */
 ConsoleTestRunner.dumpConsoleTableMessage = function(viewMessage, forceInvalidate, results) {
   if (forceInvalidate) {
@@ -237,7 +237,7 @@ ConsoleTestRunner.evaluateInConsole = function(code, callback, dontForceMainCont
 /**
  * @param {string} code
  * @param {boolean=} dontForceMainContext
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.evaluateInConsolePromise = function(code, dontForceMainContext) {
   return new Promise(fulfill => ConsoleTestRunner.evaluateInConsole(code, fulfill, dontForceMainContext));
@@ -277,14 +277,14 @@ ConsoleTestRunner.evaluateInConsoleAndDump = function(code, callback, dontForceM
 /**
  * @param {string} code
  * @param {boolean=} dontForceMainContext
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.evaluateInConsoleAndDumpPromise = function(code, dontForceMainContext) {
   return new Promise(fulfill => ConsoleTestRunner.evaluateInConsoleAndDump(code, fulfill, dontForceMainContext));
 };
 
 /**
- * @return {number}
+ * @returns {number}
  */
 ConsoleTestRunner.consoleMessagesCount = function() {
   const consoleView = Console.ConsoleView.ConsoleView.instance();
@@ -294,7 +294,7 @@ ConsoleTestRunner.consoleMessagesCount = function() {
 /**
  * @param {function(!Element):string|undefined} messageFormatter
  * @param {!Element} node
- * @return {string}
+ * @returns {string}
  */
 ConsoleTestRunner.formatterIgnoreStackFrameUrls = function(messageFormatter, node) {
   /**
@@ -321,7 +321,7 @@ ConsoleTestRunner.formatterIgnoreStackFrameUrls = function(messageFormatter, nod
 /**
  * @param {!Element} element
  * @param {!SDK.ConsoleModel.ConsoleMessage} message
- * @return {string}
+ * @returns {string}
  */
 ConsoleTestRunner.simpleFormatter = function(element, message) {
   return message.messageText + ':' + message.line + ':' + message.column;
@@ -387,7 +387,7 @@ ConsoleTestRunner.dumpConsoleClassesBrief = async function() {
 };
 
 ConsoleTestRunner.dumpConsoleCounters = async function() {
-  const counter = ConsoleCounters.WarningErrorCounter.WarningErrorCounter.instanceForTest;
+  const counter = ConsoleCounters.WarningErrorCounter.WarningErrorCounterWidget.instanceForTest;
   if (counter.updatingForTest) {
     await TestRunner.addSnifferPromise(counter, 'updatedForTest');
   }
@@ -453,7 +453,7 @@ ConsoleTestRunner.expandConsoleMessages = function(callback, deepFilter, section
 /**
  * @param {function(!Element):boolean} deepFilter
  * @param {function(!ObjectUI.ObjectPropertiesSection.ObjectPropertiesSection):boolean} sectionFilter
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.expandConsoleMessagesPromise = function(deepFilter, sectionFilter) {
   return new Promise(fulfill => ConsoleTestRunner.expandConsoleMessages(fulfill, deepFilter, sectionFilter));
@@ -519,14 +519,14 @@ ConsoleTestRunner.waitForRemoteObjectsConsoleMessages = function(callback) {
 };
 
 /**
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.waitForRemoteObjectsConsoleMessagesPromise = function() {
   return new Promise(resolve => ConsoleTestRunner.waitForRemoteObjectsConsoleMessages(resolve));
 };
 
 /**
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.waitUntilConsoleEditorLoaded = function() {
   let fulfill;
@@ -551,7 +551,7 @@ ConsoleTestRunner.waitUntilMessageReceived = function(callback) {
 };
 
 /**
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.waitUntilMessageReceivedPromise = function() {
   return new Promise(fulfill => ConsoleTestRunner.waitUntilMessageReceived(fulfill));
@@ -574,7 +574,7 @@ ConsoleTestRunner.waitUntilNthMessageReceived = function(count, callback) {
 
 /**
  * @param {number} count
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.waitUntilNthMessageReceivedPromise = function(count) {
   return new Promise(fulfill => ConsoleTestRunner.waitUntilNthMessageReceived(count, fulfill));
@@ -614,7 +614,7 @@ ConsoleTestRunner.waitForConsoleMessages = function(expectedCount, callback) {
 
 /**
  * @param {number} expectedCount
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ConsoleTestRunner.waitForConsoleMessagesPromise = async function(expectedCount) {
   await new Promise(fulfill => ConsoleTestRunner.waitForConsoleMessages(expectedCount, fulfill));
@@ -640,7 +640,7 @@ ConsoleTestRunner.selectConsoleMessages = async function(fromMessage, fromTextOf
   /**
    * @param {!Node} container
    * @param {number} offset
-   * @return {?{container: !Node, offset: number}}
+   * @returns {?{container: !Node, offset: number}}
    */
   function selectionContainerAndOffset(container, offset) {
     /** @type {?Node} */
@@ -672,7 +672,7 @@ ConsoleTestRunner.addConsoleSniffer = function(override, opt_sticky) {
 
 /**
  * @param {!Function} func
- * @return {!Function}
+ * @returns {!Function}
  */
 ConsoleTestRunner.wrapListener = function(func) {
   /**
@@ -705,7 +705,7 @@ ConsoleTestRunner.dumpStackTraces = function() {
 };
 
 /**
- * @return {!{first: number, last: number, count: number}}
+ * @returns {!{first: number, last: number, count: number}}
  */
 ConsoleTestRunner.visibleIndices = function() {
   const consoleView = Console.ConsoleView.ConsoleView.instance();

package/front_end/legacy_test_runner/coverage_test_runner/coverage_test_runner.js

@@ -8,13 +8,13 @@ import {SourcesTestRunner} from '../sources_test_runner/sources_test_runner.js';
 import {TestRunner} from '../test_runner/test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 export const CoverageTestRunner = {};
 
 /**
- * @param jsCoveragePerBlock - Collect per Block coverage if `true`, per function coverage otherwise.
- * @return {!Promise}
+ * @param jsCoveragePerBlock Collect per Block coverage if `true`, per function coverage otherwise.
+ * @returns {!Promise}
  */
 CoverageTestRunner.startCoverage = async function(jsCoveragePerBlock) {
   UI.ViewManager.ViewManager.instance().showView('coverage');
@@ -23,7 +23,7 @@ CoverageTestRunner.startCoverage = async function(jsCoveragePerBlock) {
 };
 
 /**
- * @return {!Promise}
+ * @returns {!Promise}
  */
 CoverageTestRunner.stopCoverage = function() {
   const coverageView = Coverage.CoverageView.CoverageView.instance();
@@ -31,7 +31,7 @@ CoverageTestRunner.stopCoverage = function() {
 };
 
 /**
- * @return {!Promise}
+ * @returns {!Promise}
  */
 CoverageTestRunner.suspendCoverageModel = async function() {
   const coverageView = Coverage.CoverageView.CoverageView.instance();
@@ -40,7 +40,7 @@ CoverageTestRunner.suspendCoverageModel = async function() {
 };
 
 /**
- * @return {!Promise}
+ * @returns {!Promise}
  */
 CoverageTestRunner.resumeCoverageModel = async function() {
   const coverageView = Coverage.CoverageView.CoverageView.instance();
@@ -49,7 +49,7 @@ CoverageTestRunner.resumeCoverageModel = async function() {
 };
 
 /**
- * @return {!Promise}
+ * @returns {!Promise}
  */
 CoverageTestRunner.pollCoverage = async function() {
   const coverageView = Coverage.CoverageView.CoverageView.instance();
@@ -59,7 +59,7 @@ CoverageTestRunner.pollCoverage = async function() {
 };
 
 /**
- * @return {!Promise<Coverage.CoverageModel.CoverageModel>}
+ * @returns {!Promise<Coverage.CoverageModel.CoverageModel>}
  */
 CoverageTestRunner.getCoverageModel = function() {
   const coverageView = Coverage.CoverageView.CoverageView.instance();
@@ -67,7 +67,7 @@ CoverageTestRunner.getCoverageModel = function() {
 };
 
 /**
- * @return {!Promise<string>}
+ * @returns {!Promise<string>}
  */
 CoverageTestRunner.exportReport = async function() {
   const coverageView = Coverage.CoverageView.CoverageView.instance();
@@ -82,7 +82,7 @@ CoverageTestRunner.exportReport = async function() {
 };
 
 /**
- * @return {!Promise<!SourceFrame.SourceFrame>}
+ * @returns {!Promise<!SourceFrame.SourceFrame>}
  */
 CoverageTestRunner.sourceDecorated = async function(source) {
   await UI.InspectorView.InspectorView.instance().showPanel('sources');
@@ -98,7 +98,7 @@ CoverageTestRunner.dumpDecorations = async function(source) {
 };
 
 /**
- * @return {?DataGrid.DataGridNode}
+ * @returns {?DataGrid.DataGridNode}
  */
 CoverageTestRunner.findCoverageNodeForURL = function(url) {
   const coverageListView = Coverage.CoverageView.CoverageView.instance().listView;

package/front_end/legacy_test_runner/data_grid_test_runner/data_grid_test_runner.js

@@ -5,7 +5,7 @@
 import {TestRunner} from '../test_runner/test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 export const DataGridTestRunner = {};
 

package/front_end/legacy_test_runner/device_mode_test_runner/device_mode_test_runner.js

@@ -5,7 +5,7 @@
 import * as EmulationModel from '../../models/emulation/emulation.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 export const DeviceModeTestRunner = {};
 

package/front_end/legacy_test_runner/elements_test_runner/EditDOMTestRunner.js

@@ -3,7 +3,7 @@
 // found in the LICENSE file.
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 self.ElementsTestRunner = self.ElementsTestRunner || {};
 

package/front_end/legacy_test_runner/elements_test_runner/ElementsPanelShadowSelectionOnRefreshTestRunner.js

@@ -3,7 +3,7 @@
 // found in the LICENSE file.
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 self.ElementsTestRunner = self.ElementsTestRunner || {};
 

package/front_end/legacy_test_runner/elements_test_runner/ElementsTestRunner.js

@@ -12,7 +12,7 @@ import * as Components from '../../ui/legacy/components/utils/utils.js';
 import * as UI from '../../ui/legacy/legacy.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 self.ElementsTestRunner = self.ElementsTestRunner || {};
 
@@ -30,7 +30,7 @@ ElementsTestRunner.selectNodeWithId = function(idValue, callback) {
 
 /**
  * @param {!Object} node
- * @return {!Promise.<undefined>}
+ * @returns {!Promise.<undefined>}
  */
 ElementsTestRunner.selectNode = function(node) {
   return Common.Revealer.reveal(node);
@@ -105,7 +105,7 @@ ElementsTestRunner.findNode = async function(matchFunction, callback) {
 
 /**
  * @param {function(!Element): boolean} matchFunction
- * @param {!Promise}
+ * @returns {!Promise}
  */
 ElementsTestRunner.findNodePromise = function(matchFunction) {
   return new Promise(resolve => ElementsTestRunner.findNode(matchFunction, resolve));
@@ -174,7 +174,7 @@ ElementsTestRunner.expandAndDumpEventListeners = function(eventListenersView, ca
 /**
  * @param {!EventListeners.EventListenersView.EventListenersView} eventListenersView
  * @param {boolean=} force
- * @return {!Promise}
+ * @returns {!Promise}
  */
 ElementsTestRunner.expandAndDumpEventListenersPromise = function(eventListenersView, force) {
   return new Promise(resolve => ElementsTestRunner.expandAndDumpEventListeners(eventListenersView, resolve, force));
@@ -398,7 +398,7 @@ ElementsTestRunner.selectNodeAndWaitForStylesWithComputed = function(idValue, ca
 };
 
 ElementsTestRunner.firstElementsTreeOutline = function() {
-  return Elements.ElementsPanel.ElementsPanel.instance().treeOutlines.values().next().value;
+  return Elements.ElementsPanel.ElementsPanel.instance().getTreeOutlineForTesting();
 };
 
 ElementsTestRunner.filterMatchedStyles = function(text) {
@@ -599,7 +599,7 @@ ElementsTestRunner.showEventListenersWidget = function() {
 };
 
 /**
- * @return {Promise}
+ * @returns {Promise}
  */
 ElementsTestRunner.showComputedStyles = function() {
   Elements.ElementsPanel.ElementsPanel.instance().sidebarPaneView.tabbedPane().selectTab('computed', true);
@@ -833,62 +833,6 @@ ElementsTestRunner.dumpElementsTree = function(rootNode, depth, resultsArray) {
   print((rootNode ? treeOutline.findTreeElement(rootNode) : treeOutline.rootElement()), '', depth || 10000);
 };
 
-ElementsTestRunner.dumpDOMUpdateHighlights = function(rootNode, callback, depth) {
-  let hasHighlights = false;
-  TestRunner.addSniffer(Elements.ElementsTreeOutline.ElementsTreeOutline.prototype, 'updateModifiedNodes', didUpdate);
-
-  function didUpdate() {
-    const treeOutline = ElementsTestRunner.firstElementsTreeOutline();
-    print((rootNode ? treeOutline.findTreeElement(rootNode) : treeOutline.rootElement()), '', depth || 10000);
-
-    if (!hasHighlights) {
-      TestRunner.addResult('<No highlights>');
-    }
-
-    if (callback) {
-      callback();
-    }
-  }
-
-  function print(treeItem, prefix, depth) {
-    if (!treeItem.root) {
-      const elementXPath = Elements.DOMPath.xPath(treeItem.node(), true);
-      const highlightedElements = treeItem.listItemElement.querySelectorAll('.dom-update-highlight');
-
-      for (let i = 0; i < highlightedElements.length; ++i) {
-        const element = highlightedElements[i];
-        const classList = element.classList;
-        let xpath = elementXPath;
-
-        if (classList.contains('webkit-html-attribute-name')) {
-          xpath += '/@' + element.textContent + ' (empty)';
-        } else if (classList.contains('webkit-html-attribute-value')) {
-          const name = element.parentElement.querySelector('.webkit-html-attribute-name').textContent;
-          xpath += '/@' + name + ' ' + element.textContent;
-        } else if (classList.contains('webkit-html-text-node')) {
-          xpath += '/text() "' + element.textContent + '"';
-        }
-
-        TestRunner.addResult(prefix + xpath);
-        hasHighlights = true;
-      }
-    }
-
-    if (!treeItem.expanded) {
-      return;
-    }
-
-    const children = treeItem.children();
-    const newPrefix = (treeItem.root ? '' : prefix + '    ');
-
-    for (let i = 0; depth && children && i < children.length; ++i) {
-      if (!children[i].isClosingTag || !children[i].isClosingTag()) {
-        print(children[i], newPrefix, depth - 1);
-      }
-    }
-  }
-};
-
 ElementsTestRunner.expandElementsTree = function(callback) {
   let expandedSomething = false;
   callback = TestRunner.safeWrap(callback);
@@ -1263,24 +1207,6 @@ ElementsTestRunner.dumpInspectorGridHighlightsJSON = async function(idValues, ca
   callback();
 };
 
-ElementsTestRunner.dumpInspectorDistanceJSON = function(idValue, callback) {
-  ElementsTestRunner.nodeWithId(idValue, nodeResolved);
-
-  async function nodeResolved(node) {
-    const result = await TestRunner.OverlayAgent.getHighlightObjectForTest(node.id, true);
-    const info = result['distanceInfo'];
-    if (!info) {
-      TestRunner.addResult(`${idValue}: No distance info`);
-    } else {
-      if (info['style']) {
-        info['style'] = '<style data>';
-      }
-      TestRunner.addResult(idValue + JSON.stringify(info, null, 2));
-    }
-    callback();
-  }
-};
-
 ElementsTestRunner.dumpInspectorHighlightStyleJSON = async function(idValue) {
   const node = await ElementsTestRunner.nodeWithIdPromise(idValue);
   const result = await TestRunner.OverlayAgent.getHighlightObjectForTest(node.id, false, true /* includeStyle */);

package/front_end/legacy_test_runner/elements_test_runner/SetOuterHTMLTestRunner.js

@@ -4,7 +4,7 @@
 
 import * as SDK from '../../core/sdk/sdk.js';
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 self.ElementsTestRunner = self.ElementsTestRunner || {};
 

package/front_end/legacy_test_runner/elements_test_runner/StylesUpdateLinksTestRunner.js

@@ -6,7 +6,7 @@ import * as Platform from '../../core/platform/platform.js';
 import * as Elements from '../../panels/elements/elements.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 self.ElementsTestRunner = self.ElementsTestRunner || {};
 

package/front_end/legacy_test_runner/extensions_test_runner/ExtensionsNetworkTestRunner.js

@@ -3,7 +3,7 @@
 // found in the LICENSE file.
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 
 self.extension_getRequestByUrl = function(urls, callback) {

package/front_end/legacy_test_runner/extensions_test_runner/ExtensionsTestRunner.js

@@ -4,7 +4,7 @@
 
 import * as UI from '../../ui/legacy/legacy.js';
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 self.ExtensionsTestRunner = self.ExtensionsTestRunner || {};
 

package/front_end/legacy_test_runner/heap_profiler_test_runner/heap_profiler_test_runner.js

@@ -10,7 +10,7 @@ import * as UI from '../../ui/legacy/legacy.js';
 import {TestRunner} from '../test_runner/test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 export const HeapProfilerTestRunner = {};
 
@@ -677,7 +677,7 @@ HeapProfilerTestRunner.takeAndOpenSnapshot = async function(generator, callback)
 };
 
 /**
- * @return {!Promise<!Profiler.HeapSnapshotView.HeapProfileHeader>}
+ * @returns
  */
 HeapProfilerTestRunner.takeSnapshotPromise = function() {
   return new Promise(resolve => {

package/front_end/legacy_test_runner/layers_test_runner/layers_test_runner.js

@@ -8,7 +8,7 @@ import * as Layers from '../../panels/layers/layers.js';
 import {TestRunner} from '../test_runner/test_runner.js';
 
 /**
- * @fileoverview using private properties isn't a Closure violation in tests.
+ * @file using private properties isn't a Closure violation in tests.
  */
 export const LayersTestRunner = {};