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

Unified Diff: third_party/WebKit/Source/platform/v8_inspector/V8DebuggerAgentImpl.cpp

Issue 1836653002: [DevTools] Wrap call frame with its injected script instead top injected script (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@compile-debugger-script
Patch Set: Created 4 years, 9 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/platform/v8_inspector/V8DebuggerAgentImpl.cpp
diff --git a/third_party/WebKit/Source/platform/v8_inspector/V8DebuggerAgentImpl.cpp b/third_party/WebKit/Source/platform/v8_inspector/V8DebuggerAgentImpl.cpp
index 72ada62e151e5dcc980c75b127ee4bbb1b05837c..b491f6eb2b8d518f919620e3ceba3f44fabe2f29 100644
--- a/third_party/WebKit/Source/platform/v8_inspector/V8DebuggerAgentImpl.cpp
+++ b/third_party/WebKit/Source/platform/v8_inspector/V8DebuggerAgentImpl.cpp
@@ -1000,7 +1000,7 @@ void V8DebuggerAgentImpl::evaluateOnCallFrame(ErrorString* errorString,
if (includeCommandLineAPI.fromMaybe(false) && commandLineAPI.IsEmpty())
return;
- InjectedScriptManager::ScopedGlobalObjectExtension scopeExtension(injectedScript, m_injectedScriptManager, commandLineAPI);
+ InjectedScript::ScopedGlobalObjectExtension scopeExtension(injectedScript, commandLineAPI);
v8::TryCatch tryCatch(injectedScript->isolate());
@@ -1346,15 +1346,15 @@ PassOwnPtr<Array<CallFrame>> V8DebuggerAgentImpl::currentCallFrames(ErrorString*
{
if (m_pausedContext.IsEmpty() || !m_pausedCallFrames.size())
return Array<CallFrame>::create();
- InjectedScript* injectedScript = m_injectedScriptManager->injectedScriptFor(m_pausedContext.Get(m_isolate));
- if (!injectedScript) {
+ InjectedScript* topFrameInjectedScript = m_injectedScriptManager->injectedScriptFor(m_pausedContext.Get(m_isolate));
+ if (!topFrameInjectedScript) {
// Context has been reported as removed while on pause.
return Array<CallFrame>::create();
}
- v8::Isolate* isolate = injectedScript->isolate();
+ v8::Isolate* isolate = topFrameInjectedScript->isolate();
v8::HandleScope handles(isolate);
- v8::Local<v8::Context> context = injectedScript->context();
+ v8::Local<v8::Context> context = topFrameInjectedScript->context();
v8::Context::Scope contextScope(context);
v8::Local<v8::Array> objects = v8::Array::New(isolate);
@@ -1365,7 +1365,14 @@ PassOwnPtr<Array<CallFrame>> V8DebuggerAgentImpl::currentCallFrames(ErrorString*
if (hasInternalError(errorString, details.IsEmpty()))
return Array<CallFrame>::create();
+ int contextId = currentCallFrame->contextId();
+ ErrorString ignoredErrorString;
+ InjectedScript* injectedScript = contextId ? m_injectedScriptManager->findInjectedScript(&ignoredErrorString, contextId) : nullptr;
+ if (!injectedScript)
+ injectedScript = topFrameInjectedScript;
+
String16 callFrameId = RemoteCallFrameId::serialize(injectedScript->contextId(), frameOrdinal);
+
if (hasInternalError(errorString, !details->Set(context, toV8StringInternalized(isolate, "callFrameId"), toV8String(isolate, callFrameId)).FromMaybe(false)))
return Array<CallFrame>::create();

Powered by Google App Engine
This is Rietveld 408576698