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

Unified Diff: src/inspector/debugger-script.js

Issue 2668363002: [inspector] removed unused code from JavaScriptSourceFrame (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | src/inspector/debugger_script_externs.js » ('j') | src/inspector/wasm-translation.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/inspector/debugger-script.js
diff --git a/src/inspector/debugger-script.js b/src/inspector/debugger-script.js
index c9ede086ac5b3c8bbc82faf734ea9cb28ea7062a..55da4191f6380db9e894211943636a1d56d42d23 100644
--- a/src/inspector/debugger-script.js
+++ b/src/inspector/debugger-script.js
@@ -390,8 +390,8 @@ DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror)
details = {
"functionName": ensureFuncMirror().debugName(),
"location": {
- "lineNumber": line(),
- "columnNumber": column(),
+ "lineNumber": ensureLocation().line,
+ "columnNumber": ensureLocation().column,
"scriptId": String(script.id())
},
"this": thisObject,
@@ -452,22 +452,6 @@ DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror)
/**
* @return {number}
*/
- function line()
- {
- return ensureLocation().line;
- }
-
- /**
- * @return {number}
- */
- function column()
- {
- return ensureLocation().column;
- }
-
- /**
- * @return {number}
- */
function contextId()
{
var mirror = ensureFuncMirror();
@@ -478,15 +462,6 @@ DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror)
}
/**
- * @return {number}
- */
- function sourceID()
- {
- var script = ensureScriptMirror();
- return script.id();
- }
-
- /**
* @param {string} expression
* @return {*}
*/
@@ -515,9 +490,6 @@ DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror)
}
return {
- "sourceID": sourceID,
- "line": line,
- "column": column,
"contextId": contextId,
"thisObject": thisObject,
"evaluate": evaluate,
« no previous file with comments | « no previous file | src/inspector/debugger_script_externs.js » ('j') | src/inspector/wasm-translation.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698