Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(267)

Side by Side Diff: third_party/WebKit/PerformanceTests/Canvas/resources/canvas_runner.js

Issue 2858783003: Refactor prepareToMeasureValuesAsync to startMeasureValuesAsyn which run test through callback (Closed)
Patch Set: Address Xianzhu's review comments Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // CanvasRunner is a wrapper of PerformanceTests/resources/runner.js for canvas tests. 1 // CanvasRunner is a wrapper of PerformanceTests/resources/runner.js for canvas tests.
2 (function () { 2 (function () {
3 var MEASURE_DRAW_TIMES = 50; 3 var MEASURE_DRAW_TIMES = 50;
4 var MAX_MEASURE_DRAW_TIMES = 1000; 4 var MAX_MEASURE_DRAW_TIMES = 1000;
5 var MAX_MEASURE_TIME_PER_FRAME = 1000; // 1 sec 5 var MAX_MEASURE_TIME_PER_FRAME = 1000; // 1 sec
6 var currentTest = null; 6 var currentTest = null;
7 var isTestDone = false; 7 var isTestDone = false;
8 8
9 var CanvasRunner = {}; 9 var CanvasRunner = {};
10 10
11 CanvasRunner.start = function (test) { 11 CanvasRunner.start = function (test) {
12 PerfTestRunner.prepareToMeasureValuesAsync({unit: 'runs/s', 12 PerfTestRunner.startMeasureValuesAsync({
13 description: test.description, done: testDone}); 13 unit: 'runs/s',
14 if (!test.doRun) { 14 description: test.description,
15 CanvasRunner.logFatalError("doRun must be set."); 15 done: testDone,
16 return; 16 run: function() {
17 } 17 if (!test.doRun) {
18 currentTest = test; 18 CanvasRunner.logFatalError("doRun must be set.");
19 runTest(); 19 return;
20 }
21 currentTest = test;
22 runTest();
23 }});
20 } 24 }
21 25
22 function runTest() { 26 function runTest() {
23 try { 27 try {
24 if (currentTest.preRun) 28 if (currentTest.preRun)
25 currentTest.preRun(); 29 currentTest.preRun();
26 30
27 var start = PerfTestRunner.now(); 31 var start = PerfTestRunner.now();
28 var count = 0; 32 var count = 0;
29 while ((PerfTestRunner.now() - start <= MAX_MEASURE_TIME_PER_FRAME) && (count * MEASURE_DRAW_TIMES < MAX_MEASURE_DRAW_TIMES)) { 33 while ((PerfTestRunner.now() - start <= MAX_MEASURE_TIME_PER_FRAME) && (count * MEASURE_DRAW_TIMES < MAX_MEASURE_DRAW_TIMES)) {
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 }; 89 };
86 90
87 video.addEventListener('playing', playingListener, true); 91 video.addEventListener('playing', playingListener, true);
88 video.addEventListener('timeupdate', timeupdateListener, true); 92 video.addEventListener('timeupdate', timeupdateListener, true);
89 video.loop = true; 93 video.loop = true;
90 video.play(); 94 video.play();
91 } 95 }
92 96
93 window.CanvasRunner = CanvasRunner; 97 window.CanvasRunner = CanvasRunner;
94 })(); 98 })();
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698