Store the accumulated configuration from requestMediaKeySystemAccess().

Store the accumulated configuration from requestMediaKeySystemAccess().

BUG=447059
Committed: https://crrev.com/3d1434e5dc33c5c994afd3d24e1c22b9c1d121d6
Cr-Commit-Position: refs/heads/master@{#312955}

[sandersd (OOO until July 31), 2015-01-22 22:36:41]

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

[sandersd (OOO until July 31), 2015-01-22 22:36:42]

This is the Chromium side, I believe it can be applied cleanly without the Blink
changes; then all of the Blink changes can go in together.

[sandersd (OOO until July 31), 2015-01-22 22:52:14]

Patchset #1 (id:1) has been deleted

[jrummell, 2015-01-23 00:34:32]

lgtm w/nits. I think that "getConfiguration() override;" will prevent this from
being submitted until the blink API is changed.

https://codereview.chromium.org/872453002/diff/20001/media/blink/webencrypted...
File media/blink/webencryptedmediaclient_impl.cc (right):

https://codereview.chromium.org/872453002/diff/20001/media/blink/webencrypted...
media/blink/webencryptedmediaclient_impl.cc:163: // 7.3 For each value in
supportedConfigurations, run the GetSuppored
s/GetSuppored/GetSupported/

https://codereview.chromium.org/872453002/diff/20001/media/blink/webencrypted...
media/blink/webencryptedmediaclient_impl.cc:169: // TODO(sandersd): Remove once
Blink requires the configurations parameter for
Is there any reason why this can't be removed now? Do the tests need to be
updated first?

[sandersd (OOO until July 31), 2015-01-23 20:05:52]

> I think that "getConfiguration() override;" will prevent this from being
submitted

Without 'override' it appears to compile both ways, so this can be done as a
three-sided change with a tiny third side.

https://codereview.chromium.org/872453002/diff/20001/media/blink/webencrypted...
File media/blink/webencryptedmediaclient_impl.cc (right):

https://codereview.chromium.org/872453002/diff/20001/media/blink/webencrypted...
media/blink/webencryptedmediaclient_impl.cc:163: // 7.3 For each value in
supportedConfigurations, run the GetSuppored
On 2015/01/23 00:34:32, jrummell wrote:
> s/GetSuppored/GetSupported/

Done.

https://codereview.chromium.org/872453002/diff/20001/media/blink/webencrypted...
media/blink/webencryptedmediaclient_impl.cc:169: // TODO(sandersd): Remove once
Blink requires the configurations parameter for
On 2015/01/23 00:34:32, jrummell wrote:
> Is there any reason why this can't be removed now? Do the tests need to be
> updated first?

Yes, there is one test case for this.

[sandersd (OOO until July 31), 2015-01-23 21:48:20]

The CQ bit was checked by sandersd@chromium.org

[commit-bot: I haz the power, 2015-01-23 21:49:24]

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

[commit-bot: I haz the power, 2015-01-23 22:52:41]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-01-23 22:53:34]

Patchset 3 (id:??) landed as
https://crrev.com/3d1434e5dc33c5c994afd3d24e1c22b9c1d121d6
Cr-Commit-Position: refs/heads/master@{#312955}