Changeset 96531 in webkit

Timestamp:

Oct 3, 2011 1:09:25 PM (13 years ago)

Author:

eric@webkit.org

Message:

PerformanceTests/Parser/html-parser is only testing parsing of the head element
​https://bugs.webkit.org/show_bug.cgi?id=69283

Reviewed by Adam Barth.

While investigating ​https://bugs.webkit.org/show_bug.cgi?id=68944
I found that the html-parser benchmark was only parsing up to the
first script tag per loop! We've fixed this by adding the sandbox
tag which will deny all external loads and allow the parser to
synchronously continue parsing the entire document to completion
(as we had expected it was doing).

This changes the profile somewhat. Line number counting is much
hotter, since we're actually accounting for the parse of the entire
document in our sample.

Total sample time only about doubles, from 1800ms to 3800ms on my machine
which is less than I would have expected.

• Parser/html-parser.html:

Location:

trunk/PerformanceTests

Files:

• ChangeLog (modified) (1 diff)
• Parser/html-parser.html (modified) (1 diff)

trunk/PerformanceTests/ChangeLog

@@ +1 @@
+2011-10-03  Eric Seidel  <eric@webkit.org>
+
+        PerformanceTests/Parser/html-parser is only testing parsing of the head element
+        https://bugs.webkit.org/show_bug.cgi?id=69283
+
+        Reviewed by Adam Barth.
+
+        While investigating https://bugs.webkit.org/show_bug.cgi?id=68944
+        I found that the html-parser benchmark was only parsing up to the
+        first script tag per loop!  We've fixed this by adding the sandbox
+        tag which will deny all external loads and allow the parser to
+        synchronously continue parsing the entire document to completion
+        (as we had expected it was doing).
+
+        This changes the profile somewhat.  Line number counting is much
+        hotter, since we're actually accounting for the parse of the entire
+        document in our sample.
+
+        Total sample time only about doubles, from 1800ms to 3800ms on my machine
+        which is less than I would have expected.
+
+        * Parser/html-parser.html:
+
 2011-08-09  Alexandru Chiculita  <achicu@adobe.com>
 

trunk/PerformanceTests/Parser/html-parser.html

@@ -9 +9 @@
     var iframe = document.createElement("iframe");
     iframe.src = "about:blank";
-    iframe.style.display = "none";
+    iframe.style.display = "none";  // Prevent creation of the rendering tree, so we only test HTML parsing.
+    iframe.sandbox = '';  // Prevents loading of external scripts which would otherwise pause the parser.
     document.body.appendChild(iframe);
     iframe.contentDocument.open();