MediaRecorder: add and wire read {video,audio}BitsPerSecond props

third_party/WebKit/LayoutTests/fast/mediarecorder/MediaRecorder-requestData.html

 <!DOCTYPE html>
 <script src=../../resources/testharness.js></script>
 <script src=../../resources/testharnessreport.js></script>
 <script>
 
 // This test verifies we can successfully create a MediaRecorder and
 // get data from a MediaStream with it.
 
 var checkStreamTracks = function(stream, has_video, has_audio) {
     if (has_video) {
         assert_equals(stream.getVideoTracks().length, 1);
         assert_equals(stream.getVideoTracks()[0].readyState, 'live');
     } else {
         assert_equals(stream.getVideoTracks().length, 0);
     }
 
     if (has_audio) {
         assert_equals(stream.getAudioTracks().length, 1);
         assert_equals(stream.getAudioTracks()[0].readyState, 'live');
     } else {
         assert_equals(stream.getAudioTracks().length, 0);
     }
 };
 
 var makeAsyncTest = function(value, expected) {
-    var recorder;
-
-    async_test(function() {
-        const recorderOnDataAvailable = this.step_func_done(function(event) {
+    async_test(function(test) {

[philipj_slow, 2016/02/04 03:03:10]

Using this as you had done is fine too, IMHO, but so is the new style.

[mcasas, 2016/02/05 02:39:14]

Yeah, it was a homogenisation.

+        const recorderOnDataAvailable = test.step_func_done(function(event) {
             assert_equals(event.data.size, 0,
                           'Recorded data size should be == 0');
         });
 
-        const recorderOnStop = this.step_func(function() {
+        const recorderOnStop = test.step_func(function() {
             assert_unreached('Recording stopped.');
         });
 
-        const recorderOnError = this.step_func(function() {
+        const recorderOnError = test.step_func(function() {
             assert_unreached('Recording error.');
         });
 
-        const gotStream = this.step_func_done(function(stream) {
+        const gotStream = test.step_func_done(function(stream) {
             checkStreamTracks(stream, value['video'], value['audio']);
 
-            try {
-                recorder = new MediaRecorder(stream);
-            } catch (e) {
-                assert_unreached('Exception while creating MediaRecorder: ' + e);
-            }
+            var recorder = new MediaRecorder(stream);
 
-            try {
-                recorder.requestData();
-                assert_unreached();
-            } catch (e) {
-                assert_equals(e.name, "InvalidStateError");
-            }
+            assert_throws("InvalidStateError",
+                function() {
+                    recorder.requestData();
+                },
+                "recorder throws InvalidStateError if requestData() comes in invalid state");

[philipj_slow, 2016/02/04 03:03:10]

Maybe s/comes in invalid state/while state is not "recording"/, assuming that is
what is being tested.

[mcasas, 2016/02/05 02:39:14]

Done.

 
             recorder.ondataavailable = recorderOnDataAvailable;
             recorder.onstop = recorderOnStop;
             recorder.onerror = recorderOnError;
             recorder.start();
 
             assert_equals(recorder.state, "recording");
             recorder.requestData();
         });
 
-        const onError = this.step_func(function() {
-            assert_unreached('Error creating MediaRecorder.');
-        });
-
-        try {
-            navigator.webkitGetUserMedia(value, gotStream, onError);
-        } catch(e) {
-            assert_unreached('Exception launching getUserMedia(): ' + e);
-        }
+        const onError = test.unreached_func('Error creating MediaStream.');
+        navigator.webkitGetUserMedia(value, gotStream, onError);
     });
 };
 
 generate_tests(makeAsyncTest,
                [["video-only",  {video: true,  audio: false}],
                 ["audio-only",  {video: false, audio: true}],
                 ["audio-video", {video: true,  audio: true}]]);
 
 </script>