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

Side by Side Diff: chrome/test/data/media/canvas_capture_test.html

Issue 1692773002: Canvas Capture: chrome browsertest- added frame rate test (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Excluding mac trybots for now. Created 4 years, 9 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
(Empty)
1 <!DOCTYPE html>
mcasas 2016/03/04 22:30:33 This file should go into chrome/test/data/media/ht
cpaulin (no longer in chrome) 2016/03/10 22:21:25 I have the feeling chrome/test/data/webrtc is wher
2 <html>
3 <head>
4 <title>Canvas Capture FPS Test</title>
5 </head>
6 <body>
7 <div> Canvas capture FPS test.</div>
8 <video id='local-video' muted='true' hidden loop autoplay>
9 <source src='bear-640x360.webm' type='video/webm'>
mcasas 2016/03/04 22:30:33 What's the problem with the other bear webms aroun
cpaulin (no longer in chrome) 2016/03/10 22:21:24 They don't work, I tried them on linux and mac. Wh
10 </video>
11 <br></br>
12 <label>Canvas</label>
13 <canvas id='canvas' width=640 height=360></canvas>
mcasas 2016/03/04 22:30:33 Large canvas, for a browsertests that is supposed
cpaulin (no longer in chrome) 2016/03/10 22:21:25 I will look for a smaller video. Done.
14 <br></br>
15 <label>Captured canvas stream</label>
16 <video id='captured-video' autoplay></video>
17 <script>
18
19 'use strict';
20
21 var DEFAULT_FRAME_RATE = 10;
22 var TOLERANCE_PERCENT = 15 / 100;
mcasas 2016/03/04 22:30:33 This is not a percentage, but a per_one in any ca
cpaulin (no longer in chrome) 2016/03/10 22:21:24 Done.
23
24 // Define video event listener when window is loaded.
25 window.onload = function() {
26 var video = document.getElementById('local-video');
27 var canvas = document.getElementById('canvas');
28 var context = canvas.getContext('2d');
29 video.addEventListener('play', function() {
30 draw(this, context, canvas.clientWidth, canvas.clientHeight);
31 }, false);
32 };
33
34 // This function draws the video frame on the canvas.
35 function draw(video, context, width, height) {
36 window.requestAnimationFrame(draw.bind(null, video, context, width,
37 height));
38 if (video.paused || video.ended) return false;
39 context.drawImage(video, 0, 0, width, height);
40 }
41
42 // Test class that measures the frame rate of a stream originated from
43 // canvas capture.
44 function CanvasFpsTest() {
45 this.theCanvasStream = null;
mcasas 2016/03/04 22:30:33 Inconsistent indenting: here you use 2 spaces wher
cpaulin (no longer in chrome) 2016/03/10 22:21:25 window.onload indent is a line break not a typical
46 this.decodedFrames = 0;
47 this.droppedFrames = 0;
48 this.decodedFPS = [];
49 this.droppedFPS = [];
50 this.startTime = {};
51 this.interval = {};
52 this.targetFPS = 0;
53 }
54
55 // This function starts the canvas capture of the local video stream.
56 CanvasFpsTest.prototype.startCanvasCapture = function(frameRate) {
57 var test = this;
mcasas 2016/03/04 22:30:33 Why aliasing? In l. 100+ you use |this|. Const?
cpaulin (no longer in chrome) 2016/03/10 22:21:25 Reason is this would not work: testVideo.addEventL
58 test.targetFPS = frameRate ? frameRate : DEFAULT_FRAME_RATE;
mcasas 2016/03/04 22:30:33 This function is only called by testFrameRateOfCan
cpaulin (no longer in chrome) 2016/03/10 22:21:24 I could remove the argument, it is not used in thi
59 test.theCanvasStream = document.getElementById('canvas').captureStream(
60 test.targetFPS);
61 var testVideo = document.getElementById('captured-video');
62 testVideo.addEventListener('play', function(event) {
63 test.startTime = new Date().getTime();
64 test.interval = setInterval(test.calculateStats.bind(test, testVideo),
65 1000);
66 });
67 testVideo.src = URL.createObjectURL(test.theCanvasStream);
68 };
69
70 // This function calculates the FPS for decoded frames and dropped frames.
71 CanvasFpsTest.prototype.calculateStats = function(video) {
72 // We calculate stats on 6 seconds worth of video, that should be
73 // reasonable.
74 if (this.decodedFPS.length == 6) {
mcasas 2016/03/04 22:30:33 Make a const out of this, put it on top of file?
cpaulin (no longer in chrome) 2016/03/10 22:21:25 Done.
75 clearInterval(this.interval);
76 return this.validateResults();
77 }
78 if (video.readyState <= HTMLMediaElement.HAVE_CURRENT_DATA ||
79 video.paused || video.ended)
80 return;
mcasas 2016/03/04 22:30:33 Uncertain about this but: {} around? (due to multi
cpaulin (no longer in chrome) 2016/03/10 22:21:24 Done.
81 var currentTime = new Date().getTime();
82 var deltaTime = (currentTime - this.startTime) / 1000;
83
84 this.startTime = currentTime;
85
86 // Calculate decoded frames per second.
87 var fps = (video.webkitDecodedFrameCount - this.decodedFrames) / deltaTime;
88 this.decodedFrames = video.webkitDecodedFrameCount;
89 this.decodedFPS.push(fps);
90
91 // Calculate dropped frames per second.
92 fps = (video.webkitDroppedFrameCount - this.droppedFrames) / deltaTime;
93 this.droppedFrames = video.webkitDroppedFrameCount;
94 this.droppedFPS.push(fps);
95 };
96
97 // This function validates the frame rate against accepted error tolerance.
98 CanvasFpsTest.prototype.validateResults = function() {
99 // Pop out the first value as it could be a bit off.
100 this.decodedFPS.shift;
101 this.droppedFPS.shift;
102 var averageFPS = average(this.decodedFPS);
103 var averageDroppedFPS = average(this.droppedFPS);
104 var errorFPS = Math.abs(averageFPS - this.targetFPS) / this.targetFPS;
105 var minFPS = this.targetFPS * (1 - TOLERANCE_PERCENT);
106 var maxFPS = this.targetFPS * (1 + TOLERANCE_PERCENT);
107 console.log('Average dropped FPS:', averageDroppedFPS);
108 if (errorFPS > TOLERANCE_PERCENT) {
109 reportResult('ERROR: Expected FPS between ' + minFPS + ' and ' + maxFPS +
110 ' (' + 100 * TOLERANCE_PERCENT + '% tolerance) but got ' + averageFPS +
111 ' instead');
112 } else {
113 reportResult('OK');
114 }
115 };
116
117 // This function calculates average of array values.
118 function average(array) {
119 var count = array.length;
120 var total = 0;
121 for (var i = 0; i < count; i++) {
mcasas 2016/03/04 22:30:33 for (value i in array) ... https://google.githu
cpaulin (no longer in chrome) 2016/03/10 22:21:25 Done.
122 total += parseInt(array[i]);
123 }
124 return Math.floor(total / count);
125 }
126
127 // This function logs and reports result to Test Environment.
128 function reportResult(message) {
129 console.log(message);
130 if (!window.domAutomationController)
131 throw 'Expected to run in an automated context.';
132 window.domAutomationController.send(message);
133 }
134
135 // Tests that canvas capture will approximately honor the given frame rate.
136 function testFrameRateOfCanvasCapture(frameRate) {
137 var fpsTest = new CanvasFpsTest();
138 fpsTest.startCanvasCapture(frameRate);
139 }
140
141 </script>
142 </body>
143 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698