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

Unified Diff: Source/bindings/v8/DebuggerScript.js

Issue 466243002: Support merged Dart-JS callstacks (Closed) Base URL: svn://svn.chromium.org/blink/branches/dart/dartium
Patch Set: ptal Created 6 years, 3 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/bindings/v8/DebuggerScript.js
diff --git a/Source/bindings/v8/DebuggerScript.js b/Source/bindings/v8/DebuggerScript.js
index 6973cf6979e689692351885792a66a9681c22f36..bb85fad490adad427b5c3745606010fea9523e2b 100644
--- a/Source/bindings/v8/DebuggerScript.js
+++ b/Source/bindings/v8/DebuggerScript.js
@@ -351,6 +351,7 @@ DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror, callerFrame, sc
var thisObject = frameDetails.receiver();
var isAtReturn = !!frameDetails.isAtReturn();
+ var cachedFramePointer = frameDetails.framePointer();
var returnValue = isAtReturn ? frameDetails.returnValue() : undefined;
var scopeMirrors = (scopeDetailsLevel === DebuggerScript.ScopeInfoDetails.NoScopes ? [] : frameMirror.allScopes(scopeDetailsLevel === DebuggerScript.ScopeInfoDetails.FastAsyncScopes));
@@ -434,6 +435,10 @@ DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror, callerFrame, sc
return scriptId;
}
+ function framePointer() {
+ return cachedFramePointer;
+ }
+
function functionName()
{
// FIXMEDART: remove caching when Dart devtools refactor CL lands.
@@ -501,6 +506,7 @@ DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror, callerFrame, sc
"setVariableValue": setVariableValue,
"stepInPositions": stepInPositions,
"isAtReturn": isAtReturn,
+ "framePointer": framePointer,
"returnValue": returnValue
};
}

Powered by Google App Engine
This is Rietveld 408576698