Improve container detection testing by adding additional containers

Improve container detection testing by testing additional containers (new containers added https://codereview.chromium.org/16034004/). Also put back original reference URLs in source.

BUG=235108
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203019

[jrummell, 2013-05-29 18:27:35]

PTAL. Improve container detection testing by adding additional containers in the
test directory (all generated from bear.ogv using avconv). Also put back
original reference URLs in source.

[xhwang, 2013-05-29 18:41:50]

LGTM

Could you please update the README about how you generated these files? (the
command you uses etc)

Also, you probably need to commit the binaries first otherwise the trybots will
fail.

[xhwang, 2013-05-29 18:43:22]

https://codereview.chromium.org/15966005/diff/1/media/base/container_names_un...
File media/base/container_names_unittest.cc (right):

https://codereview.chromium.org/15966005/diff/1/media/base/container_names_un...
media/base/container_names_unittest.cc:114: // Test several OGG files to ensure
that the container is detected properly.
These comments don't add much value as the tests themselves are pretty clear
about what we are doing. Remove?

[xhwang, 2013-05-29 18:46:29]

Please also update the CL description with the correct BUG number. I found that
the BUG number (238108) in your last CL
(https://codereview.chromium.org/14495010/) is wrong.

[jrummell, 2013-05-29 20:46:39]

New binary files have been checked in separately. This is now just updating the
unit tests (plus change URLs in the original code).

https://codereview.chromium.org/15966005/diff/1/media/base/container_names_un...
File media/base/container_names_unittest.cc (right):

https://codereview.chromium.org/15966005/diff/1/media/base/container_names_un...
media/base/container_names_unittest.cc:114: // Test several OGG files to ensure
that the container is detected properly.
On 2013/05/29 18:43:22, xhwang wrote:
> These comments don't add much value as the tests themselves are pretty clear
> about what we are doing. Remove?

Done.

[commit-bot: I haz the power, 2013-05-29 20:51:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jrummell@chromium.org/15966005/5001

[commit-bot: I haz the power, 2013-05-30 00:20:00]

Change committed as 203019