Add html-parser-threaded perf test now that the threaded parser is enabled by default
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Mar 2013 19:07:26 +0000 (19:07 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Mar 2013 19:07:26 +0000 (19:07 +0000)
https://bugs.webkit.org/show_bug.cgi?id=111200

Reviewed by Ryosuke Niwa.

PerformanceTests:

The old srcdoc test no longer triggers the threaded parser due to compatibility fixes.

* Parser/html-parser-threaded.html: Renamed from PerformanceTests/Parser/html-parser-srcdoc.html.
* Skipped: Unfortunately this new test is still skipped too.

Tools:

Had to teach run-perf-tests to ignore one more warning.

* Scripts/webkitpy/performance_tests/perftest.py:
(PerfTest):

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

PerformanceTests/ChangeLog
PerformanceTests/Parser/html-parser-threaded.html [moved from PerformanceTests/Parser/html-parser-srcdoc.html with 90% similarity]
PerformanceTests/Skipped
Tools/ChangeLog
Tools/Scripts/webkitpy/performance_tests/perftest.py

index 9d8f979..073c4cf 100644 (file)
@@ -1,3 +1,15 @@
+2013-03-05  Eric Seidel  <eric@webkit.org>
+
+        Add html-parser-threaded perf test now that the threaded parser is enabled by default
+        https://bugs.webkit.org/show_bug.cgi?id=111200
+
+        Reviewed by Ryosuke Niwa.
+
+        The old srcdoc test no longer triggers the threaded parser due to compatibility fixes.
+
+        * Parser/html-parser-threaded.html: Renamed from PerformanceTests/Parser/html-parser-srcdoc.html.
+        * Skipped: Unfortunately this new test is still skipped too.
+
 2013-03-04  Eric Seidel  <eric@webkit.org>
 
         Add PerformanceTest for document startup via iframe append/remove
@@ -6,7 +6,7 @@
 if (window.internals && window.internals.settings.setThreadedHTMLParser)
     window.internals.settings.setThreadedHTMLParser(true);
 
-var spec = PerfTestRunner.loadFile("resources/html5.html");
+var specURL = "resources/html5.html";
 
 var iframe = document.createElement("iframe");
 iframe.style.display = "none";  // Prevent creation of the rendering tree, so we only test HTML parsing.
@@ -19,10 +19,10 @@ iframe.onload = function() {
     var now = PerfTestRunner.now();
     PerfTestRunner.measureValueAsync(now - then);
     then = now;
-    iframe.srcdoc = spec;
+    iframe.src = specURL;
 }
 var then = PerfTestRunner.now();
-iframe.srcdoc = spec;
+iframe.src = specURL;
 
 function onCompletedRun() {
     iframe.onload = null;
index 0433571..e752cba 100644 (file)
@@ -71,7 +71,7 @@ ShadowDOM
 
 Canvas/drawimage.html
 
-# Works fine, but is mostly useful to the threaded-parsing folks, as
-# it's very similar to parser/html-parser.html just using
-# srcdoc to load instead of document.write.
-Parser/html-parser-srcdoc.html
+# Has a time-dependency (monotonically increasing), and is much slower than
+# necessary due to https://bugs.webkit.org/show_bug.cgi?id=107236
+# but is still useful for testing the threaded parser.
+Parser/html-parser-threaded.html
index 1e7b976..9da3035 100644 (file)
@@ -1,3 +1,15 @@
+2013-03-05  Eric Seidel  <eric@webkit.org>
+
+        Add html-parser-threaded perf test now that the threaded parser is enabled by default
+        https://bugs.webkit.org/show_bug.cgi?id=111200
+
+        Reviewed by Ryosuke Niwa.
+
+        Had to teach run-perf-tests to ignore one more warning.
+
+        * Scripts/webkitpy/performance_tests/perftest.py:
+        (PerfTest):
+
 2013-03-05  Jochen Eisinger  <jochen@chromium.org>
 
         Update list of builders for test results server
index 45c50c1..d53e7f1 100644 (file)
@@ -246,6 +246,7 @@ class PerfTest(object):
         # Following is for html5.html
         re.compile(re.escape("""Blocked access to external URL http://www.whatwg.org/specs/web-apps/current-work/""")),
         re.compile(r"CONSOLE MESSAGE: (line \d+: )?Blocked script execution in '[A-Za-z0-9\-\.:]+' because the document's frame is sandboxed and the 'allow-scripts' permission is not set."),
+        re.compile(r"CONSOLE MESSAGE: (line \d+: )?Not allowed to load local resource"),
         # Dromaeo reports values for subtests. Ignore them for now.
         re.compile(r'(?P<name>.+): \[(?P<values>(\d+(.\d+)?,\s+)*\d+(.\d+)?)\]'),
     ]