Report support for the flac codec in ogg containers.

Report support for the flac codec in ogg containers.

Per official documentation this is a supported combination:
https://wiki.xiph.org/Ogg#Codecs

BUG=689569
TEST=updated canPlayType tests.

Review-Url: https://codereview.chromium.org/2681683004
Cr-Commit-Position: refs/heads/master@{#449816}
Committed: https://chromium.googlesource.com/chromium/src/+/7f7dca6bddebc85000891307c03d6e3af106646b

[DaleCurtis, 2017-02-07 20:41:08]

dalecurtis@chromium.org changed reviewers:
+ ddorwin@chromium.org

[ddorwin, 2017-02-08 01:41:10]

LGTM. Do we have playback tests? We should add some since we are now advertising
support on all platforms.

https://codereview.chromium.org/2681683004/diff/1/content/browser/media/media...
File content/browser/media/media_canplaytype_browsertest.cc (right):

https://codereview.chromium.org/2681683004/diff/1/content/browser/media/media...
content/browser/media/media_canplaytype_browsertest.cc:694:
CanPlay("'application/ogg; codecs=\"theora, opus\"'"));
These two are out of order (compared to the rest).

[DaleCurtis, 2017-02-10 23:17:46]

The CQ bit was checked by dalecurtis@chromium.org

[DaleCurtis, 2017-02-10 23:17:46]

Thanks for review, added a playback test.

https://codereview.chromium.org/2681683004/diff/1/content/browser/media/media...
File content/browser/media/media_canplaytype_browsertest.cc (right):

https://codereview.chromium.org/2681683004/diff/1/content/browser/media/media...
content/browser/media/media_canplaytype_browsertest.cc:694:
CanPlay("'application/ogg; codecs=\"theora, opus\"'"));
On 2017/02/08 at 01:41:10, ddorwin wrote:
> These two are out of order (compared to the rest).

Done.

[DaleCurtis, 2017-02-10 23:17:47]

The patchset sent to the CQ was uploaded after l-g-t-m from ddorwin@chromium.org
Link to the patchset: https://codereview.chromium.org/2681683004/#ps20001
(title: "Add playback test.")

[commit-bot: I haz the power, 2017-02-10 23:18:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ddorwin, 2017-02-10 23:39:50]

lgtm

[commit-bot: I haz the power, 2017-02-11 01:18:46]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486768666261930,
"parent_rev": "c0e5d3dd86889273b81949ad73f2a0994fbfb4cf", "commit_rev":
"7f7dca6bddebc85000891307c03d6e3af106646b"}

[commit-bot: I haz the power, 2017-02-11 01:20:22]

Description was changed from

==========
Report support for the flac codec in ogg containers.

Per official documentation this is a supported combination:
https://wiki.xiph.org/Ogg#Codecs

BUG=689569
TEST=updated canPlayType tests.
==========

to

==========
Report support for the flac codec in ogg containers.

Per official documentation this is a supported combination:
https://wiki.xiph.org/Ogg#Codecs

BUG=689569
TEST=updated canPlayType tests.

Review-Url: https://codereview.chromium.org/2681683004
Cr-Commit-Position: refs/heads/master@{#449816}
Committed:
https://chromium.googlesource.com/chromium/src/+/7f7dca6bddebc85000891307c03d...
==========

[commit-bot: I haz the power, 2017-02-11 01:20:23]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7f7dca6bddebc85000891307c03d...