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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js

Issue 2857453002: DevTools: support resolving a UILocation to multiple raw script locations (Closed)
Patch Set: Review comments addressed Created 3 years, 8 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: third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js b/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js
index 78bfe1a2f98b64d5d9df2b14f09be29c6d825980..0aeaca3d1a5a081fdcaa75b253762dcbdb7887d5 100644
--- a/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js
+++ b/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js
@@ -1088,8 +1088,15 @@ SDK.DebuggerModel.Location = class {
/**
* @return {string}
*/
+ uniqueScriptId() {
+ return this.debuggerModel.target().id() + ':' + this.scriptId;
+ }
+
+ /**
+ * @return {string}
+ */
id() {
- return this.debuggerModel.target().id() + ':' + this.scriptId + ':' + this.lineNumber + ':' + this.columnNumber;
+ return this.uniqueScriptId() + ':' + this.lineNumber + ':' + this.columnNumber;
}
};

Powered by Google App Engine
This is Rietveld 408576698