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

Unified Diff: Source/devtools/front_end/sdk/UISourceCode.js

Issue 464963002: DevTools: Make UISourceCode Target-agnostic (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 months 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: Source/devtools/front_end/sdk/UISourceCode.js
diff --git a/Source/devtools/front_end/sdk/UISourceCode.js b/Source/devtools/front_end/sdk/UISourceCode.js
index 78830cdbe258784693beac3a02411e577f3bc0be..9f1bd534e2c803689f33e601323aa7f870685745 100644
--- a/Source/devtools/front_end/sdk/UISourceCode.js
+++ b/Source/devtools/front_end/sdk/UISourceCode.js
@@ -53,16 +53,6 @@ WebInspector.UISourceCode = function(project, parentPath, name, originURL, url,
/** @type {!Array.<!WebInspector.PresentationConsoleMessage>} */
this._consoleMessages = [];
- /**
- * @type {!Map.<!WebInspector.Target, !WebInspector.SourceMapping>}
- */
- this._sourceMappingForTarget = new Map();
-
- /**
- * @type {!Map.<!WebInspector.Target, !WebInspector.ResourceScriptFile>}
- */
- this._scriptFileForTarget = new Map();
-
/** @type {!Array.<!WebInspector.Revision>} */
this.history = [];
}
@@ -223,27 +213,6 @@ WebInspector.UISourceCode.prototype = {
},
/**
- * @param {!WebInspector.Target} target
- * @return {?WebInspector.ResourceScriptFile}
- */
- scriptFileForTarget: function(target)
- {
- return this._scriptFileForTarget.get(target) || null;
- },
-
- /**
- * @param {!WebInspector.Target} target
- * @param {?WebInspector.ResourceScriptFile} scriptFile
- */
- setScriptFileForTarget: function(target, scriptFile)
- {
- if (scriptFile)
- this._scriptFileForTarget.put(target, scriptFile);
- else
- this._scriptFileForTarget.remove(target);
- },
-
- /**
* @return {!WebInspector.Project}
*/
project: function()
@@ -606,52 +575,6 @@ WebInspector.UISourceCode.prototype = {
},
/**
- * @param {!WebInspector.Target} target
- * @param {number} lineNumber
- * @param {number} columnNumber
- * @return {?WebInspector.RawLocation}
- */
- uiLocationToRawLocation: function(target, lineNumber, columnNumber)
- {
- var sourceMapping = this._sourceMappingForTarget.get(target);
- if (!sourceMapping)
- return null;
- return sourceMapping.uiLocationToRawLocation(this, lineNumber, columnNumber);
- },
-
- /**
- * @param {number} lineNumber
- * @param {number} columnNumber
- * @return {!Array.<!WebInspector.RawLocation>}
- */
- uiLocationToRawLocations: function(lineNumber, columnNumber)
- {
- var result = [];
- var sourceMappings = this._sourceMappingForTarget.values();
- for (var i = 0; i < sourceMappings.length; ++i) {
- var rawLocation = sourceMappings[i].uiLocationToRawLocation(this, lineNumber, columnNumber);
- if (rawLocation)
- result.push(rawLocation);
- }
- return result;
- },
-
- /**
- * @param {number} lineNumber
- * @return {boolean}
- */
- uiLineHasMapping: function(lineNumber)
- {
- var sourceMappings = this._sourceMappingForTarget.values();
- for (var i = 0; i < sourceMappings.length; ++i) {
- var sourceMapping = sourceMappings[i];
- if (!sourceMappings[i].uiLineHasMapping(this, lineNumber))
- return false;
- }
- return true;
- },
-
- /**
* @return {!Array.<!WebInspector.PresentationConsoleMessage>}
*/
consoleMessages: function()
@@ -684,31 +607,6 @@ WebInspector.UISourceCode.prototype = {
},
/**
- * @return {boolean}
- */
- hasSourceMapping: function()
- {
- return !!this._sourceMappingForTarget.size();
- },
-
- /**
- * @param {!WebInspector.Target} target
- * @param {?WebInspector.SourceMapping} sourceMapping
- */
- setSourceMappingForTarget: function(target, sourceMapping)
- {
- if (this._sourceMappingForTarget.get(target) === sourceMapping)
- return;
-
- if (sourceMapping)
- this._sourceMappingForTarget.put(target, sourceMapping);
- else
- this._sourceMappingForTarget.remove(target);
-
- this.dispatchEventToListeners(WebInspector.UISourceCode.Events.SourceMappingChanged, {target: target, isIdentity: sourceMapping ? sourceMapping.isIdentity() : false});
- },
-
- /**
* @param {number} lineNumber
* @param {number=} columnNumber
* @return {!WebInspector.UILocation}
@@ -738,23 +636,6 @@ WebInspector.UILocation = function(uiSourceCode, lineNumber, columnNumber)
WebInspector.UILocation.prototype = {
/**
- * @param {!WebInspector.Target} target
- * @return {?WebInspector.RawLocation}
- */
- uiLocationToRawLocation: function(target)
- {
- return this.uiSourceCode.uiLocationToRawLocation(target, this.lineNumber, this.columnNumber);
- },
-
- /**
- * @return {!Array.<!WebInspector.RawLocation>}
- */
- uiLocationToRawLocations: function()
- {
- return this.uiSourceCode.uiLocationToRawLocations(this.lineNumber, this.columnNumber);
- },
-
- /**
* @return {string}
*/
linkText: function()

Powered by Google App Engine
This is Rietveld 408576698