Apply script preprocessor to Web page scripts only.

Source/core/inspector/PageDebuggerAgent.cpp

Index: Source/core/inspector/PageDebuggerAgent.cpp
diff --git a/Source/core/inspector/PageDebuggerAgent.cpp b/Source/core/inspector/PageDebuggerAgent.cpp
index 72bd983efa425e8384062f4341f8320a1a9c427d..b0977d5e3e2cc090369a700333e2084b46fc54de 100644
--- a/Source/core/inspector/PageDebuggerAgent.cpp
+++ b/Source/core/inspector/PageDebuggerAgent.cpp
@@ -33,9 +33,11 @@
 
 #include "bindings/v8/DOMWrapperWorld.h"
 #include "bindings/v8/PageScriptDebugServer.h"
+#include "bindings/v8/ScriptController.h"
 #include "core/inspector/InspectorOverlay.h"
 #include "core/inspector/InspectorPageAgent.h"
 #include "core/inspector/InstrumentingAgents.h"
+#include "core/page/Frame.h"
 #include "core/page/Page.h"
 #include "core/page/PageConsole.h"
 
@@ -136,7 +138,9 @@ void PageDebuggerAgent::didClearWindowObjectInWorld(Frame* frame, DOMWrapperWorl
         return;
 
     reset();
-    scriptDebugServer().setScriptPreprocessor(m_pageAgent->scriptPreprocessor());
+    frame->script()->clearScriptPreprocessor();

[abarth-chromium, 2013/07/09 23:14:10]

Please clear the script preprocessor in ScriptController.  The ScriptController
should have enough knowledge to understand that it's not leaking the world by
holding the script preprocessor.  We shouldn't have to reason able the
PageDebuggerAgent to come to that conclusion.

[johnjbarton, 2013/07/10 23:24:23]

Sorry I don't understand how to do this.

I assume we would need to have a call into a ScriptController function every
time that didClearWindowObjectInWorld() is called and we would have to know that
such a call always precedes the PageDebuggerAgent call (to avoid clearing valid
preprocessor sources).  Do we have such a function? 

+    if (!m_pageAgent->scriptPreprocessor().isEmpty())
+        frame->script()->setScriptPreprocessor(m_pageAgent->scriptPreprocessor());
 }
 
 } // namespace WebCore