Changeset 60684 in webkit

Timestamp:

Jun 4, 2010 9:59:23 AM (14 years ago)

Author:

eric@webkit.org

Message:

2010-06-04 Tony Gentilcore <​tonyg@chromium.org>

Reviewed by Adam Barth.

Persist V8's ScriptData to the memory cache.
​https://bugs.webkit.org/show_bug.cgi?id=38661

This stores V8's ScriptData in the memory cache and also causes the
network platform layer to be notified of the available cacheable
metadata.

Chromium's morejs benchmark shows a 3-4% improvement on fast hardware.

No new tests because no new functionality.

• bindings/v8/ScriptSourceCode.h: (WebCore::ScriptSourceCode::ScriptSourceCode): (WebCore::ScriptSourceCode::cachedScript):
• bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::compileScript): (WebCore::V8Proxy::precompileScript): (WebCore::V8Proxy::evaluate):
• bindings/v8/V8Proxy.h:

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• bindings/v8/ScriptSourceCode.h (modified) (5 diffs)
• bindings/v8/V8Proxy.cpp (modified) (6 diffs)
• bindings/v8/V8Proxy.h (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-06-04  Tony Gentilcore  <tonyg@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        Persist V8's ScriptData to the memory cache.
+        https://bugs.webkit.org/show_bug.cgi?id=38661
+
+        This stores V8's ScriptData in the memory cache and also causes the
+        network platform layer to be notified of the available cacheable
+        metadata.
+
+        Chromium's morejs benchmark shows a 3-4% improvement on fast hardware.
+
+        No new tests because no new functionality.
+
+        * bindings/v8/ScriptSourceCode.h:
+        (WebCore::ScriptSourceCode::ScriptSourceCode):
+        (WebCore::ScriptSourceCode::cachedScript):
+        * bindings/v8/V8Proxy.cpp:
+        (WebCore::V8Proxy::compileScript):
+        (WebCore::V8Proxy::precompileScript):
+        (WebCore::V8Proxy::evaluate):
+        * bindings/v8/V8Proxy.h:
+
 2010-06-04  Tony Gentilcore  <tonyg@chromium.org>
 

trunk/WebCore/bindings/v8/ScriptSourceCode.h

@@ -32 +32 @@
 #define ScriptSourceCode_h
 
+#include "CachedResourceHandle.h"
 #include "CachedScript.h"
 #include "KURL.h"
@@ -42 +43 @@
     ScriptSourceCode(const String& source, const KURL& url = KURL(), int startLine = 1)
         : m_source(source)
+        , m_cachedScript(0)
         , m_url(url)
         , m_startLine(startLine)
@@ -51 +53 @@
     ScriptSourceCode(CachedScript* cs)
         : m_source(cs->script())
+        , m_cachedScript(cs)
         , m_url(ParsedURLString, cs->url())
         , m_startLine(1)
@@ -59 +62 @@
 
     const String& source() const { return m_source; }
+    CachedScript* cachedScript() const { return m_cachedScript.get(); }
     const KURL& url() const { return m_url; }
     int startLine() const { return m_startLine; }
@@ -64 +68 @@
 private:
     String m_source;
+    CachedResourceHandle<CachedScript> m_cachedScript;
     KURL m_url;
     int m_startLine;

trunk/WebCore/bindings/v8/V8Proxy.cpp

@@ -32 +32 @@
 #include "V8Proxy.h"
 
+#include "CachedMetadata.h"
 #include "CSSMutableStyleDeclaration.h"
 #include "DateExtension.h"
@@ -71 +72 @@
 #include <wtf/Assertions.h>
 #include <wtf/OwnArrayPtr.h>
+#include <wtf/OwnPtr.h>
 #include <wtf/StdLibExtras.h>
 #include <wtf/StringExtras.h>
@@ -234 +236 @@
 }
 
-v8::Handle<v8::Script> V8Proxy::compileScript(v8::Handle<v8::String> code, const String& fileName, int baseLine)
+v8::Handle<v8::Script> V8Proxy::compileScript(v8::Handle<v8::String> code, const String& fileName, int baseLine, v8::ScriptData* scriptData)
 {
     const uint16_t* fileNameString = fromWebCoreString(fileName);
@@ -240 +242 @@
     v8::Handle<v8::Integer> line = v8::Integer::New(baseLine);
     v8::ScriptOrigin origin(name, line);
-    v8::Handle<v8::Script> script = v8::Script::Compile(code, &origin);
+    v8::Handle<v8::Script> script = v8::Script::Compile(code, &origin, scriptData);
     return script;
 }
@@ -339 +341 @@
 }
 
+PassOwnPtr<v8::ScriptData> V8Proxy::precompileScript(const ScriptSourceCode& source)
+{
+    // A pseudo-randomly chosen ID used to store and retrieve V8 ScriptData from
+    // the CachedScript. If the format changes, this ID should be changed too.
+    static const unsigned dataTypeID = 0xECC13BD7;
+
+    // Very small scripts are not worth the effort to preparse.
+    static const unsigned minPreparseLength = 1024;
+
+    CachedScript* cachedScript = source.cachedScript();
+    if (!cachedScript)
+        return 0;
+
+    CachedMetadata* cachedMetadata = cachedScript->cachedMetadata(dataTypeID);
+    if (cachedMetadata)
+        return v8::ScriptData::New(cachedMetadata->data(), cachedMetadata->size());
+
+    const CString& utf8Source = source.source().utf8();
+    if (utf8Source.length() < minPreparseLength)
+        return 0;
+
+    v8::ScriptData* scriptData = v8::ScriptData::PreCompile(utf8Source.data(), utf8Source.length());
+    cachedScript->setCachedMetadata(dataTypeID, scriptData->Data(), scriptData->Length());
+
+    return scriptData;
+}
+
 v8::Local<v8::Value> V8Proxy::evaluate(const ScriptSourceCode& source, Node* node)
 {
@@ -364 +393 @@
         PlatformBridge::traceEventBegin("v8.compile", node, "");
 #endif
+        OwnPtr<v8::ScriptData> scriptData = precompileScript(source);
 
         // NOTE: For compatibility with WebCore, ScriptSourceCode's line starts at
         // 1, whereas v8 starts at 0.
-        v8::Handle<v8::Script> script = compileScript(code, source.url(), source.startLine() - 1);
+        v8::Handle<v8::Script> script = compileScript(code, source.url(), source.startLine() - 1, scriptData.get());
 #if PLATFORM(CHROMIUM)
         PlatformBridge::traceEventEnd("v8.compile", node, "");

trunk/WebCore/bindings/v8/V8Proxy.h

@@ -286 +286 @@
         static v8::Handle<v8::Value> checkNewLegal(const v8::Arguments&);
 
-        static v8::Handle<v8::Script> compileScript(v8::Handle<v8::String> code, const String& fileName, int baseLine);
+        static v8::Handle<v8::Script> compileScript(v8::Handle<v8::String> code, const String& fileName, int baseLine, v8::ScriptData* scriptData = 0);
 
         // If the exception code is different from zero, a DOM exception is
@@ -338 +338 @@
         void resetIsolatedWorlds();
 
+        PassOwnPtr<v8::ScriptData> precompileScript(const ScriptSourceCode& source);
+
         // Returns false when we're out of memory in V8.
         bool setInjectedScriptContextDebugId(v8::Handle<v8::Context> targetContext);