DevTools: Runtime.compileScript with persistScript = false should not report script

third_party/WebKit/Source/platform/v8_inspector/V8RuntimeAgentImpl.cpp

Index: third_party/WebKit/Source/platform/v8_inspector/V8RuntimeAgentImpl.cpp
diff --git a/third_party/WebKit/Source/platform/v8_inspector/V8RuntimeAgentImpl.cpp b/third_party/WebKit/Source/platform/v8_inspector/V8RuntimeAgentImpl.cpp
index cf2c2ba82e78c37f908e18bc1283ad9a97fd7e2d..fe7557d0b949148d937b87fc663c00f9bf9a1b2a 100644
--- a/third_party/WebKit/Source/platform/v8_inspector/V8RuntimeAgentImpl.cpp
+++ b/third_party/WebKit/Source/platform/v8_inspector/V8RuntimeAgentImpl.cpp
@@ -37,6 +37,7 @@
 #include "platform/v8_inspector/RemoteObjectId.h"
 #include "platform/v8_inspector/V8ConsoleMessage.h"
 #include "platform/v8_inspector/V8Debugger.h"
+#include "platform/v8_inspector/V8DebuggerAgentImpl.h"
 #include "platform/v8_inspector/V8InspectorImpl.h"
 #include "platform/v8_inspector/V8InspectorSessionImpl.h"
 #include "platform/v8_inspector/V8StackTraceImpl.h"
@@ -503,7 +504,12 @@ void V8RuntimeAgentImpl::compileScript(ErrorString* errorString,
     if (!scope.initialize())
         return;
 
+    V8DebuggerAgentImpl* debuggerAgent = m_session->debuggerAgent();
+    if (!persistScript)
+        debuggerAgent->setIgnoreScriptParsedEvents(true);

[dgozman, 2016/08/09 20:43:08]

Remote the indirection, call into V8Debugger here.

[lushnikov, 2016/08/09 22:24:29]

Done.

     v8::Local<v8::Script> script = m_inspector->compileScript(scope.context(), toV8String(m_inspector->isolate(), expression), sourceURL, false);
+    if (!persistScript)
+        debuggerAgent->setIgnoreScriptParsedEvents(false);
     if (script.IsEmpty()) {
         v8::Local<v8::Message> message = scope.tryCatch().Message();
         if (!message.IsEmpty())