Title: [199267] trunk/PerformanceTests
Revision
199267
Author
[email protected]
Date
2016-04-08 22:16:59 -0700 (Fri, 08 Apr 2016)

Log Message

Have Animometer benchmark always start with complexity of 1
https://bugs.webkit.org/show_bug.cgi?id=156432

Reviewed by Ryosuke Niwa.

* Animometer/tests/resources/main.js: Update the default Controller and RampController to
set its minimum complexities to 1 instead of 0.

Modified Paths

Diff

Modified: trunk/PerformanceTests/Animometer/tests/resources/main.js (199266 => 199267)


--- trunk/PerformanceTests/Animometer/tests/resources/main.js	2016-04-09 04:56:42 UTC (rev 199266)
+++ trunk/PerformanceTests/Animometer/tests/resources/main.js	2016-04-09 05:16:59 UTC (rev 199267)
@@ -53,7 +53,7 @@
         // Length of subsequent intervals; a value of 0 means use no intervals
         this.intervalSamplingLength = 100;
 
-        this.initialComplexity = 0;
+        this.initialComplexity = 1;
     }, {
 
     set isFrameLengthEstimatorEnabled(enabled) {
@@ -327,8 +327,8 @@
         this._tier = -.5;
         // The timestamp is first set after the first interval completes
         this._tierStartTimestamp = 0;
-        this._minimumComplexity = 0;
-        this._maximumComplexity = 0;
+        this._minimumComplexity = 1;
+        this._maximumComplexity = 1;
 
         // After the tier range is determined, figure out the number of ramp iterations
         var minimumRampLength = 2500;
@@ -409,7 +409,7 @@
             this._endTimestamp += timestamp;
             this.mark(Strings.json.samplingStartTimeOffset, timestamp);
 
-            this._minimumComplexity = 0;
+            this._minimumComplexity = 1;
             this._possibleMinimumComplexity = this._minimumComplexity;
             this._minimumComplexityEstimator.sample(this._minimumComplexity);
 
@@ -468,7 +468,7 @@
         } else if (intervalFrameLengthStandardDeviation > 2) {
             // In the case where we might have found a previous interval where 60fps was reached. We hit a significant blip,
             // so we should resample this area in the next ramp.
-            this._possibleMinimumComplexity = 0;
+            this._possibleMinimumComplexity = 1;
         }
         if (intervalFrameLengthMean - intervalFrameLengthStandardDeviation > this.frameLengthRampLowerThreshold)
             this._possibleMaximumComplexity = Math.min(this._possibleMaximumComplexity, currentComplexity);
@@ -505,7 +505,7 @@
         this._rampDidWarmup = false;
         // Start timestamp represents start of ramp iteration and warm up
         this._rampStartTimestamp = timestamp;
-        this._possibleMinimumComplexity = 0;
+        this._possibleMinimumComplexity = 1;
         this._possibleMaximumComplexity = this._maximumComplexity;
     },
 

Modified: trunk/PerformanceTests/ChangeLog (199266 => 199267)


--- trunk/PerformanceTests/ChangeLog	2016-04-09 04:56:42 UTC (rev 199266)
+++ trunk/PerformanceTests/ChangeLog	2016-04-09 05:16:59 UTC (rev 199267)
@@ -1,5 +1,15 @@
 2016-04-08  Jon Lee  <[email protected]>
 
+        Have Animometer benchmark always start with complexity of 1
+        https://bugs.webkit.org/show_bug.cgi?id=156432
+
+        Reviewed by Ryosuke Niwa.
+
+        * Animometer/tests/resources/main.js: Update the default Controller and RampController to
+        set its minimum complexities to 1 instead of 0.
+
+2016-04-08  Jon Lee  <[email protected]>
+
         Fix SVG benchmark test
         https://bugs.webkit.org/show_bug.cgi?id=156410
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to