Apply script preprocessor to Web page scripts only.

Source/bindings/v8/ScriptController.h

 /*
  * Copyright (C) 2008, 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 46 matching lines @@
 class V8WindowShell;
 class Widget;
 
 typedef WTF::Vector<v8::Extension*> V8Extensions;
 
 enum ReasonForCallingCanExecuteScripts {
     AboutToExecuteScript,
     NotAboutToExecuteScript
 };
 
+enum InternalIsoloatedWorldIds {
+    ScriptPreprocessorIsolatedWorldId = 10001,

[abarth-chromium, 2013/07/08 23:57:08]

Why is it safe to steal this world ID?  It's valid for the embedder to use this
world API itself, which would cause a collision.

[johnjbarton, 2013/07/09 22:44:54]

It is not absolutely safe, only relatively safe given that the only known
embedder using world ids starts at 1 and increments by 1 in a session unlikely
to include thousands of worlds.

Let's discuss how to solve this on:
https://code.google.com/p/chromium/issues/detail?id=258504

+};
+
 class ScriptController {
 public:
     ScriptController(Frame*);
     ~ScriptController();
 
     bool initializeMainWorld();
     V8WindowShell* windowShell(DOMWrapperWorld*);
     V8WindowShell* existingWindowShell(DOMWrapperWorld*);
 
     ScriptValue executeScript(const ScriptSourceCode&);
@@ skipping 73 matching lines @@
     NPObject* createScriptObjectForPluginElement(HTMLPlugInElement*);
     NPObject* windowScriptNPObject();
     NPP frameNPP();
 
     // Registers a v8 extension to be available on webpages. Will only
     // affect v8 contexts initialized after this call. Takes ownership of
     // the v8::Extension object passed.
     static void registerExtensionIfNeeded(v8::Extension*);
     static V8Extensions& registeredExtensions();
 
+    // Empty string disables preprocessing.

[abarth-chromium, 2013/07/08 23:57:08]

Rather than using in-band signaling, I'd recommend a separate
clearScriptPreprocessor() function.

[johnjbarton, 2013/07/09 22:44:54]

Done.

+    void setScriptPreprocessor(const String& preprocessorSource);
+    // Source to Source processing iff debugger enabled and preprocessor loaded.
+    String preprocess(const String& scriptSource, const String& scriptName);
+    bool hasScriptPreprocessor();
+    bool isScriptPreprocessing();

[abarth-chromium, 2013/07/08 23:57:08]

isPreprocessingScript

[johnjbarton, 2013/07/09 22:44:54]

Done.

+
     bool setContextDebugId(int);
     static int contextDebugId(v8::Handle<v8::Context>);
 
 private:
     typedef HashMap<int, OwnPtr<V8WindowShell> > IsolatedWorldMap;
 
     NPObject* createScriptObjectForFrame();
     void clearForClose(bool destroyGlobal);
 
     Frame* m_frame;
     const String* m_sourceURL;
     v8::Isolate* m_isolate;
 
     OwnPtr<V8WindowShell> m_windowShell;
     IsolatedWorldMap m_isolatedWorlds;
 
     bool m_paused;
 
     NPObject* m_windowScriptNPObject;
     OwnPtr<NPP_t> m_frameNPP;
 
     typedef HashSet<NPP> PluginInstances;
     PluginInstances m_pluginNPPs;
+
+    class ScriptPreprocessor;
+    String m_preprocessorSource;
+    OwnPtr<ScriptPreprocessor> m_scriptPreprocessor;
 };
 
 } // namespace WebCore
 
 #endif // ScriptController_h