PerformanceTests/Parser/html-parser is only testing parsing of the head element
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 3 Oct 2011 20:09:25 +0000 (20:09 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 3 Oct 2011 20:09:25 +0000 (20:09 +0000)
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:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@96531 268f45cc-cd09-0410-ab3c-d52691b4dbfc

PerformanceTests/ChangeLog
PerformanceTests/Parser/html-parser.html

index 89c63029369e686171b613587bdf09c85f8c6e3f..3eec52e7fccd39f71e28bff2f725cd7450b02270 100644 (file)
@@ -1,3 +1,26 @@
+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>
 
         Fixing 65868 REGRESSION(r92610) caused by 65668 - Optimize floating elements lookup
index bb2c73c4bc57a16a19460b6feb93e1357e3edf59..bf713028358ca8f97e6f4358604350b9af1b3905 100644 (file)
@@ -8,7 +8,8 @@ var spec = loadFile("resources/html5.html");
 start(20, function() {
     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();
     iframe.contentDocument.write(spec);