PerformanceTests/CSS/StyleSheetInsert.html has a time-dependent non-Gaussian distribution
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 27 Sep 2012 23:39:09 +0000 (23:39 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 27 Sep 2012 23:39:09 +0000 (23:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=97741

Reviewed by Antti Koivisto.

Reset the content in the iframe in each run so that the runtime from each run follows a normal distribution.
We can cleanup this code once my patch to replace the argument list of run by an object is landed.

* CSS/StyleSheetInsert.html:

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

PerformanceTests/CSS/StyleSheetInsert.html
PerformanceTests/ChangeLog

index a1fa082..90a5f34 100644 (file)
@@ -7,19 +7,28 @@
 <iframe></iframe>
 </body>
 <script>
-var frame = document.getElementsByTagName("iframe")[0];
-var testDoc = frame.contentDocument;
-var docText = "";
-docText += "<body><style>.foo {color:red}</style>";
-docText += "<div class='bar'>Foo</div>";
-for (var i = 0; i < 10000; ++i)
-    docText += "<div class='foo'>Foo</div>";
-testDoc.body.innerHTML = docText;
+
+function setup() {
+    var frame = document.getElementsByTagName("iframe")[0];
+    var testDoc = frame.contentDocument;
+    var docText = "";
+    docText += "<body><style>.foo {color:red}</style>";
+    docText += "<div class='bar'>Foo</div>";
+    for (var i = 0; i < 10000; ++i)
+        docText += "<div class='foo'>Foo</div>";
+    testDoc.body.innerHTML = docText;
+    return testDoc;
+}
 
 PerfTestRunner.run(function() {
-    var styleElem = testDoc.createElement("style");
-    styleElem.innerText = ".bar {color:green}";
-    testDoc.body.insertBefore(styleElem, testDoc.body.firstChild);
-}, 50, 10);
+    var testDoc = setup();
+    var start = PerfTestRunner.now();
+    for (var i = 0; i < 50; i++) {
+        var styleElem = testDoc.createElement("style");
+        styleElem.innerText = ".bar {color:green}";
+        testDoc.body.insertBefore(styleElem, testDoc.body.firstChild);
+    }
+    return PerfTestRunner.now() - start;
+}, 1);
 </script>
 </html>
index c25a2ae..55930ae 100644 (file)
@@ -1,5 +1,17 @@
 2012-09-27  Ryosuke Niwa  <rniwa@webkit.org>
 
+        PerformanceTests/CSS/StyleSheetInsert.html has a time-dependent non-Gaussian distribution
+        https://bugs.webkit.org/show_bug.cgi?id=97741
+
+        Reviewed by Antti Koivisto.
+
+        Reset the content in the iframe in each run so that the runtime from each run follows a normal distribution.
+        We can cleanup this code once my patch to replace the argument list of run by an object is landed.
+
+        * CSS/StyleSheetInsert.html:
+
+2012-09-27  Ryosuke Niwa  <rniwa@webkit.org>
+
         Unreviewed. Minor appearance tweaks after r129813.
 
         * resources/results-template.html: