Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(354)

Unified Diff: third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js

Issue 2466123002: DevTools: reformat front-end code to match chromium style. (Closed)
Patch Set: all done Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js
diff --git a/third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js b/third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js
index 4efbc4eb527eeffb218685e6f8d9efbb13da5f5d..862c8a0d8f85574ce4ec8a4d1f22998bd6c6e0de 100644
--- a/third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js
+++ b/third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js
@@ -1,198 +1,186 @@
// Copyright 2014 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.
-
/**
- * @constructor
- * @param {!WebInspector.TargetManager} targetManager
- * @param {!WebInspector.Workspace} workspace
- * @param {!WebInspector.FileSystemWorkspaceBinding} fileSystemWorkspaceBinding
- * @param {!WebInspector.FileSystemMapping} fileSystemMapping
+ * @unrestricted
*/
-WebInspector.NetworkMapping = function(targetManager, workspace, fileSystemWorkspaceBinding, fileSystemMapping)
-{
+WebInspector.NetworkMapping = class {
+ /**
+ * @param {!WebInspector.TargetManager} targetManager
+ * @param {!WebInspector.Workspace} workspace
+ * @param {!WebInspector.FileSystemWorkspaceBinding} fileSystemWorkspaceBinding
+ * @param {!WebInspector.FileSystemMapping} fileSystemMapping
+ */
+ constructor(targetManager, workspace, fileSystemWorkspaceBinding, fileSystemMapping) {
this._targetManager = targetManager;
this._workspace = workspace;
this._fileSystemWorkspaceBinding = fileSystemWorkspaceBinding;
this._fileSystemMapping = fileSystemMapping;
- InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.RevealSourceLine, this._revealSourceLine, this);
+ InspectorFrontendHost.events.addEventListener(
+ InspectorFrontendHostAPI.Events.RevealSourceLine, this._revealSourceLine, this);
var fileSystemManager = fileSystemWorkspaceBinding.fileSystemManager();
this._eventListeners = [
- fileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemAdded, this._fileSystemAdded, this),
- fileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemRemoved, this._fileSystemRemoved, this),
+ fileSystemManager.addEventListener(
+ WebInspector.IsolatedFileSystemManager.Events.FileSystemAdded, this._fileSystemAdded, this),
+ fileSystemManager.addEventListener(
+ WebInspector.IsolatedFileSystemManager.Events.FileSystemRemoved, this._fileSystemRemoved, this),
];
- fileSystemManager.waitForFileSystems()
- .then(this._fileSystemsLoaded.bind(this));
-};
-
-WebInspector.NetworkMapping.prototype = {
- /**
- * @param {!Array<!WebInspector.IsolatedFileSystem>} fileSystems
- */
- _fileSystemsLoaded: function(fileSystems)
- {
- for (var fileSystem of fileSystems)
- this._addMappingsForFilesystem(fileSystem);
- },
-
- /**
- * @param {!WebInspector.Event} event
- */
- _fileSystemAdded: function(event)
- {
- var fileSystem = /** @type {!WebInspector.IsolatedFileSystem} */ (event.data);
- this._addMappingsForFilesystem(fileSystem);
- },
-
- /**
- * @param {!WebInspector.IsolatedFileSystem} fileSystem
- */
- _addMappingsForFilesystem: function(fileSystem)
- {
- this._addingFileSystem = true;
- this._fileSystemMapping.addFileSystem(fileSystem.path());
-
- var mappings = fileSystem.projectProperty("mappings");
- for (var i = 0; Array.isArray(mappings) && i < mappings.length; ++i) {
- var mapping = mappings[i];
- if (!mapping || typeof mapping !== "object")
- continue;
- var folder = mapping["folder"];
- var url = mapping["url"];
- if (typeof folder !== "string" || typeof url !== "string")
- continue;
- this._fileSystemMapping.addNonConfigurableFileMapping(fileSystem.path(), url, folder);
- }
- this._addingFileSystem = false;
- },
+ fileSystemManager.waitForFileSystems().then(this._fileSystemsLoaded.bind(this));
+ }
+
+ /**
+ * @param {!Array<!WebInspector.IsolatedFileSystem>} fileSystems
+ */
+ _fileSystemsLoaded(fileSystems) {
+ for (var fileSystem of fileSystems)
+ this._addMappingsForFilesystem(fileSystem);
+ }
+
+ /**
+ * @param {!WebInspector.Event} event
+ */
+ _fileSystemAdded(event) {
+ var fileSystem = /** @type {!WebInspector.IsolatedFileSystem} */ (event.data);
+ this._addMappingsForFilesystem(fileSystem);
+ }
+
+ /**
+ * @param {!WebInspector.IsolatedFileSystem} fileSystem
+ */
+ _addMappingsForFilesystem(fileSystem) {
+ this._addingFileSystem = true;
+ this._fileSystemMapping.addFileSystem(fileSystem.path());
+
+ var mappings = fileSystem.projectProperty('mappings');
+ for (var i = 0; Array.isArray(mappings) && i < mappings.length; ++i) {
+ var mapping = mappings[i];
+ if (!mapping || typeof mapping !== 'object')
+ continue;
+ var folder = mapping['folder'];
+ var url = mapping['url'];
+ if (typeof folder !== 'string' || typeof url !== 'string')
+ continue;
+ this._fileSystemMapping.addNonConfigurableFileMapping(fileSystem.path(), url, folder);
+ }
+ this._addingFileSystem = false;
+ }
+
+ /**
+ * @param {!WebInspector.Event} event
+ */
+ _fileSystemRemoved(event) {
+ var fileSystem = /** @type {!WebInspector.IsolatedFileSystem} */ (event.data);
+ this._fileSystemMapping.removeFileSystem(fileSystem.path());
+ }
+
+ /**
+ * @param {!WebInspector.Target} target
+ * @param {?WebInspector.ResourceTreeFrame} frame
+ * @param {string} url
+ * @return {?WebInspector.UISourceCode}
+ */
+ _networkUISourceCodeForURL(target, frame, url) {
+ return this._workspace.uiSourceCode(WebInspector.NetworkProject.projectId(target, frame, false), url);
+ }
+
+ /**
+ * @param {!WebInspector.Target} target
+ * @param {?WebInspector.ResourceTreeFrame} frame
+ * @param {string} url
+ * @return {?WebInspector.UISourceCode}
+ */
+ _contentScriptUISourceCodeForURL(target, frame, url) {
+ return this._workspace.uiSourceCode(WebInspector.NetworkProject.projectId(target, frame, true), url);
+ }
+
+ /**
+ * @param {!WebInspector.Target} target
+ * @param {?WebInspector.ResourceTreeFrame} frame
+ * @param {string} url
+ * @return {?WebInspector.UISourceCode}
+ */
+ _uiSourceCodeForURL(target, frame, url) {
+ return this._networkUISourceCodeForURL(target, frame, url) ||
+ this._contentScriptUISourceCodeForURL(target, frame, url);
+ }
+
+ /**
+ * @param {string} url
+ * @param {!WebInspector.Script} script
+ * @return {?WebInspector.UISourceCode}
+ */
+ uiSourceCodeForScriptURL(url, script) {
+ var frame = WebInspector.ResourceTreeFrame.fromScript(script);
+ return this._uiSourceCodeForURL(script.target(), frame, url);
+ }
+
+ /**
+ * @param {string} url
+ * @param {!WebInspector.CSSStyleSheetHeader} header
+ * @return {?WebInspector.UISourceCode}
+ */
+ uiSourceCodeForStyleURL(url, header) {
+ var frame = WebInspector.ResourceTreeFrame.fromStyleSheet(header);
+ return this._uiSourceCodeForURL(header.target(), frame, url);
+ }
+
+ /**
+ * @param {string} url
+ * @return {?WebInspector.UISourceCode}
+ */
+ uiSourceCodeForURLForAnyTarget(url) {
+ return WebInspector.workspace.uiSourceCodeForURL(url);
+ }
+
+ /**
+ * @param {!WebInspector.UISourceCode} networkUISourceCode
+ * @param {!WebInspector.UISourceCode} uiSourceCode
+ */
+ addMapping(networkUISourceCode, uiSourceCode) {
+ var fileSystemPath = WebInspector.FileSystemWorkspaceBinding.fileSystemPath(uiSourceCode.project().id());
+ this._fileSystemMapping.addMappingForResource(networkUISourceCode.url(), fileSystemPath, uiSourceCode.url());
+ }
+
+ /**
+ * @param {!WebInspector.UISourceCode} uiSourceCode
+ */
+ removeMapping(uiSourceCode) {
+ this._fileSystemMapping.removeMappingForURL(uiSourceCode.url());
+ }
+
+ /**
+ * @param {!WebInspector.Event} event
+ */
+ _revealSourceLine(event) {
+ var url = /** @type {string} */ (event.data['url']);
+ var lineNumber = /** @type {number} */ (event.data['lineNumber']);
+ var columnNumber = /** @type {number} */ (event.data['columnNumber']);
+
+ var uiSourceCode = this.uiSourceCodeForURLForAnyTarget(url);
+ if (uiSourceCode) {
+ WebInspector.Revealer.reveal(uiSourceCode.uiLocation(lineNumber, columnNumber));
+ return;
+ }
/**
* @param {!WebInspector.Event} event
+ * @this {WebInspector.NetworkMapping}
*/
- _fileSystemRemoved: function(event)
- {
- var fileSystem = /** @type {!WebInspector.IsolatedFileSystem} */ (event.data);
- this._fileSystemMapping.removeFileSystem(fileSystem.path());
- },
-
- /**
- * @param {!WebInspector.Target} target
- * @param {?WebInspector.ResourceTreeFrame} frame
- * @param {string} url
- * @return {?WebInspector.UISourceCode}
- */
- _networkUISourceCodeForURL: function(target, frame, url)
- {
- return this._workspace.uiSourceCode(WebInspector.NetworkProject.projectId(target, frame, false), url);
- },
-
- /**
- * @param {!WebInspector.Target} target
- * @param {?WebInspector.ResourceTreeFrame} frame
- * @param {string} url
- * @return {?WebInspector.UISourceCode}
- */
- _contentScriptUISourceCodeForURL: function(target, frame, url)
- {
- return this._workspace.uiSourceCode(WebInspector.NetworkProject.projectId(target, frame, true), url);
- },
-
- /**
- * @param {!WebInspector.Target} target
- * @param {?WebInspector.ResourceTreeFrame} frame
- * @param {string} url
- * @return {?WebInspector.UISourceCode}
- */
- _uiSourceCodeForURL: function(target, frame, url)
- {
- return this._networkUISourceCodeForURL(target, frame, url) || this._contentScriptUISourceCodeForURL(target, frame, url);
- },
-
- /**
- * @param {string} url
- * @param {!WebInspector.Script} script
- * @return {?WebInspector.UISourceCode}
- */
- uiSourceCodeForScriptURL: function(url, script)
- {
- var frame = WebInspector.ResourceTreeFrame.fromScript(script);
- return this._uiSourceCodeForURL(script.target(), frame, url);
- },
-
- /**
- * @param {string} url
- * @param {!WebInspector.CSSStyleSheetHeader} header
- * @return {?WebInspector.UISourceCode}
- */
- uiSourceCodeForStyleURL: function(url, header)
- {
- var frame = WebInspector.ResourceTreeFrame.fromStyleSheet(header);
- return this._uiSourceCodeForURL(header.target(), frame, url);
- },
-
- /**
- * @param {string} url
- * @return {?WebInspector.UISourceCode}
- */
- uiSourceCodeForURLForAnyTarget: function(url)
- {
- return WebInspector.workspace.uiSourceCodeForURL(url);
- },
-
- /**
- * @param {!WebInspector.UISourceCode} networkUISourceCode
- * @param {!WebInspector.UISourceCode} uiSourceCode
- */
- addMapping: function(networkUISourceCode, uiSourceCode)
- {
- var fileSystemPath = WebInspector.FileSystemWorkspaceBinding.fileSystemPath(uiSourceCode.project().id());
- this._fileSystemMapping.addMappingForResource(networkUISourceCode.url(), fileSystemPath, uiSourceCode.url());
- },
+ function listener(event) {
+ var uiSourceCode = /** @type {!WebInspector.UISourceCode} */ (event.data);
+ if (uiSourceCode.url() === url) {
+ WebInspector.Revealer.reveal(uiSourceCode.uiLocation(lineNumber, columnNumber));
+ this._workspace.removeEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, listener, this);
+ }
+ }
- /**
- * @param {!WebInspector.UISourceCode} uiSourceCode
- */
- removeMapping: function(uiSourceCode)
- {
- this._fileSystemMapping.removeMappingForURL(uiSourceCode.url());
- },
+ this._workspace.addEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, listener, this);
+ }
- /**
- * @param {!WebInspector.Event} event
- */
- _revealSourceLine: function(event)
- {
- var url = /** @type {string} */ (event.data["url"]);
- var lineNumber = /** @type {number} */ (event.data["lineNumber"]);
- var columnNumber = /** @type {number} */ (event.data["columnNumber"]);
-
- var uiSourceCode = this.uiSourceCodeForURLForAnyTarget(url);
- if (uiSourceCode) {
- WebInspector.Revealer.reveal(uiSourceCode.uiLocation(lineNumber, columnNumber));
- return;
- }
-
- /**
- * @param {!WebInspector.Event} event
- * @this {WebInspector.NetworkMapping}
- */
- function listener(event)
- {
- var uiSourceCode = /** @type {!WebInspector.UISourceCode} */ (event.data);
- if (uiSourceCode.url() === url) {
- WebInspector.Revealer.reveal(uiSourceCode.uiLocation(lineNumber, columnNumber));
- this._workspace.removeEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, listener, this);
- }
- }
-
- this._workspace.addEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, listener, this);
- },
-
- dispose: function()
- {
- WebInspector.EventTarget.removeEventListeners(this._eventListeners);
- }
+ dispose() {
+ WebInspector.EventTarget.removeEventListeners(this._eventListeners);
+ }
};
/**

Powered by Google App Engine
This is Rietveld 408576698