MediaRecorderHandler and WebmMuxer: add support for H264.

media/muxers/webm_muxer_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/macros.h"
@@ skipping 225 matching lines @@
 }
 
 const kTestParams kTestCases[] = {
     // TODO: consider not enumerating every combination by hand.
     {kCodecVP8, 1 /* num_video_tracks */, 0 /*num_audio_tracks*/},
     {kCodecVP8, 0, 1},
     {kCodecVP8, 1, 1},
     {kCodecVP9, 1, 0},
     {kCodecVP9, 0, 1},
     {kCodecVP9, 1, 1},
+    {kCodecH264, 1, 0},
+    {kCodecH264, 0, 1},
+    {kCodecH264, 1, 1},

[emircan, 2016/04/25 19:26:16]

Good time to adress TODO:
const VideoCodec kCodecs[] = {kCodecVP8, kCodecVP9, kCodecH264};
testing::Combine(testing::ValuesIn(kCodecs), testing::Values(0, 1),
testing::Values(0, 1))

However, I see you are skipping (0,0). Can that pass automatically? 

[mcasas, 2016/04/26 18:13:38]

I tried for a while but it turned out to be
involved because of the need to skip the (0,0)
case: it hits a DCHECK in Webm ctor [1], so
I'd just leave it like this.

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/media/muxers/webm_...

 };
 
 INSTANTIATE_TEST_CASE_P(, WebmMuxerTest, ValuesIn(kTestCases));
 
 }  // namespace media