Check that containers are supported by Chrome in requestMediaKeySystemAccess().

Check that containers are supported by Chrome in requestMediaKeySystemAccess().

This can only fail if a CDM declares support for a container that Chrome does not, which would be a bug in CDM registration.

BUG=374751
Committed: https://crrev.com/571587c351897e2d4053959b8e4b988d96fed9aa
Cr-Commit-Position: refs/heads/master@{#322486}

[sandersd (OOO until July 31), 2015-03-26 17:50:39]

sandersd@chromium.org changed reviewers:
+ ddorwin@chromium.org, jrummell@chromium.org

[ddorwin, 2015-03-26 17:55:10]

Please update the commit message to use the full method name. Only a few people
are familiar with the acronym, and it's not useful when searching commit
messages.

https://codereview.chromium.org/1035923004/diff/1/media/blink/webencryptedmed...
File media/blink/webencryptedmediaclient_impl.cc (right):

https://codereview.chromium.org/1035923004/diff/1/media/blink/webencryptedmed...
media/blink/webencryptedmediaclient_impl.cc:150: if
(!net::IsSupportedMediaMimeType(container_lower))
This would allow, for example, "video/ogg", which is not supported encrypted.
Why doesn't IsSupportedCodecCombination() check the container when codec_vector
is empty?

[sandersd (OOO until July 31), 2015-03-26 18:11:05]

https://codereview.chromium.org/1035923004/diff/1/media/blink/webencryptedmed...
File media/blink/webencryptedmediaclient_impl.cc (right):

https://codereview.chromium.org/1035923004/diff/1/media/blink/webencryptedmed...
media/blink/webencryptedmediaclient_impl.cc:150: if
(!net::IsSupportedMediaMimeType(container_lower))
On 2015/03/26 17:55:10, ddorwin wrote:
> This would allow, for example, "video/ogg", which is not supported encrypted.
> Why doesn't IsSupportedCodecCombination() check the container when
codec_vector
> is empty?

IsSupportedCodecCombination() already makes sure that the CDM supports the
container, so nothing exotic should show up here.

IsSupportedStrictMediaMimeType() returns Maybe when the container requires a
codec list and none is given (but also in other cases we don't support).

[ddorwin, 2015-03-26 20:55:15]

lgtm

[sandersd (OOO until July 31), 2015-03-26 22:04:00]

The CQ bit was checked by sandersd@chromium.org

[commit-bot: I haz the power, 2015-03-26 22:04:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1035923004/20001

[commit-bot: I haz the power, 2015-03-26 22:52:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-26 22:53:57]

Patchset 2 (id:??) landed as
https://crrev.com/571587c351897e2d4053959b8e4b988d96fed9aa
Cr-Commit-Position: refs/heads/master@{#322486}