Store the loaded file locally in measurePageLoadTime
authorzoltan@webkit.org <zoltan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Oct 2012 06:43:28 +0000 (06:43 +0000)
committerzoltan@webkit.org <zoltan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Oct 2012 06:43:28 +0000 (06:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=100172

Reviewed by Csaba Osztrogon√°c.

Store the loaded file as a local variable instead of attaching it to the test object.
Handle chunkSize inside test.run.

* resources/runner.js:
(.):

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

PerformanceTests/ChangeLog
PerformanceTests/resources/runner.js

index 617bce4..7f2aba9 100644 (file)
@@ -1,5 +1,18 @@
 2012-10-23  Zoltan Horvath  <zoltan@webkit.org>
 
+        Store the loaded file locally in measurePageLoadTime
+        https://bugs.webkit.org/show_bug.cgi?id=100172
+
+        Reviewed by Csaba Osztrogon√°c.
+
+        Store the loaded file as a local variable instead of attaching it to the test object.
+        Handle chunkSize inside test.run.
+
+        * resources/runner.js:
+        (.):
+
+2012-10-23  Zoltan Horvath  <zoltan@webkit.org>
+
         Add measurePageLoadTime function to PerfTestRunner
         https://bugs.webkit.org/show_bug.cgi?id=100029
 
index 6df3b9f..278c176 100755 (executable)
@@ -306,18 +306,22 @@ if (window.testRunner) {
 
     PerfTestRunner.measurePageLoadTime = function(test) {
         test.run = function() {
+            var file = PerfTestRunner.loadFile(test.path);
+            if (!test.chunkSize)
+                this.chunkSize = 50000;
+
             var chunks = [];
             // The smaller the chunks the more style resolves we do.
             // Smaller chunk sizes will show more samples in style resolution.
             // Larger chunk sizes will show more samples in line layout.
             // Smaller chunk sizes run slower overall, as the per-chunk overhead is high.
-            var chunkCount = Math.ceil(this.file.length / this.chunkSize);
+            var chunkCount = Math.ceil(file.length / this.chunkSize);
             for (var chunkIndex = 0; chunkIndex < chunkCount; chunkIndex++) {
-                var chunk = this.file.substr(chunkIndex * this.chunkSize, this.chunkSize);
+                var chunk = file.substr(chunkIndex * this.chunkSize, this.chunkSize);
                 chunks.push(chunk);
             }
 
-            PerfTestRunner.logInfo("Testing " + this.file.length + " byte document in " + chunkCount + " " + this.chunkSize + " byte chunks.");
+            PerfTestRunner.logInfo("Testing " + file.length + " byte document in " + chunkCount + " " + this.chunkSize + " byte chunks.");
 
             var iframe = document.createElement("iframe");
             document.body.appendChild(iframe);
@@ -341,10 +345,6 @@ if (window.testRunner) {
             document.body.removeChild(iframe);
         };
 
-        this.file = PerfTestRunner.loadFile(test.path);
-        if (!test.chunkSize)
-            this.chunkSize = 50000;
-
         PerfTestRunner.measureTime(test);
     }