DevTools: make InjectedScript heap-allocated.

third_party/WebKit/Source/core/inspector/InjectedScript.h

Index: third_party/WebKit/Source/core/inspector/InjectedScript.h
diff --git a/third_party/WebKit/Source/core/inspector/InjectedScript.h b/third_party/WebKit/Source/core/inspector/InjectedScript.h
index 05315254c3652796ace771577b51e96942eceb0d..2aa4e4847ca26e8021f07de69dce3ab5ded3262f 100644
--- a/third_party/WebKit/Source/core/inspector/InjectedScript.h
+++ b/third_party/WebKit/Source/core/inspector/InjectedScript.h
@@ -48,14 +48,11 @@ class ScriptFunctionCall;
 typedef String ErrorString;
 PassRefPtr<JSONValue> toJSONValue(const ScriptValue&);
 
-
 class InjectedScript final {
-    DISALLOW_NEW();

[sof, 2016/01/24 08:05:32]

USING_FAST_MALLOC(InjectedScript); ?

[sof, 2016/01/25 18:59:12]

Not following up on this one?

[pfeldman, 2016/01/25 19:02:41]

I absolutely have to! Why allocating this rare object slowly when there is a way
to speed things up!

 public:
     InjectedScript();
     ~InjectedScript();
 
-    bool isEmpty() const { return m_injectedScriptObject.isEmpty(); }
     ScriptState* scriptState() const
     {
         ASSERT(!isEmpty());
@@ -117,7 +114,7 @@ public:
     int contextId() { return m_contextId; }
 
 private:
-    friend InjectedScript InjectedScriptManager::injectedScriptFor(ScriptState*);
+    friend InjectedScript* InjectedScriptManager::injectedScriptFor(ScriptState*);
     using InspectedStateAccessCheck = bool (*)(ScriptState*);
     InjectedScript(ScriptValue, InspectedStateAccessCheck, PassRefPtr<InjectedScriptNative>, int contextId);