html5-full-render.html test uses substring but meant substr
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Oct 2012 06:50:45 +0000 (06:50 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Oct 2012 06:50:45 +0000 (06:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=99495

Patch by Elliott Sprehn <esprehn@chromium.org> on 2012-10-16
Reviewed by Adam Barth.

This test intended to chunk up the HTML5 spec into chunks of 500k chars, but used
substring(to,from) instead of substr(from,length) so it actually loaded 5x more
content than was in the HTML5 spec.

* Parser/html5-full-render.html:

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

PerformanceTests/ChangeLog
PerformanceTests/Parser/html5-full-render.html

index d636f6c..67c124e 100644 (file)
@@ -1,3 +1,16 @@
+2012-10-16  Elliott Sprehn  <esprehn@chromium.org>
+
+        html5-full-render.html test uses substring but meant substr
+        https://bugs.webkit.org/show_bug.cgi?id=99495
+
+        Reviewed by Adam Barth.
+
+        This test intended to chunk up the HTML5 spec into chunks of 500k chars, but used
+        substring(to,from) instead of substr(from,length) so it actually loaded 5x more
+        content than was in the HTML5 spec.
+
+        * Parser/html5-full-render.html:
+
 2012-10-11  James Simonsen  <simonjam@chromium.org>
 
         unprefix window.performance.webkitNow()
index b59e44c..9426057 100644 (file)
@@ -13,7 +13,7 @@ var chunks = [];
 var chunkSize = 500000; // 6.09mb / 500k = approx 13 chunks (thus 13 forced layouts/style resolves).
 var chunkCount = Math.ceil(spec.length / chunkSize);
 for (var chunkIndex = 0; chunkIndex < chunkCount; chunkIndex++) {
-    var chunk = spec.substring(chunkIndex * chunkSize, chunkSize);
+    var chunk = spec.substr(chunkIndex * chunkSize, chunkSize);
     chunks.push(chunk);
 }