[Chromecast] Make EME persistent license support optional

[Chromecast] Make EME persistent license support optional

The recent refactoring from KeySystemsInfo meant we lost the option
to control whether persistent license support is enabled (see linked
bug).  Add the option back into the new system.

BUG=internal b/28844432

Committed: https://crrev.com/73a640d498ea36d6c60daca10c5a2b7979511660
Cr-Commit-Position: refs/heads/master@{#397827}

[halliwell, 2016-06-02 03:16:24]

Description was changed from

==========
[Chromecast] Make EME persistent license support optional

The recent refactoring from KeySystemsInfo meant we lost the option
to control whether persistent license support is enabled (see linked
bug).  Add the option back into the new system.

BUG=internal b/28844432
==========

to

==========
[Chromecast] Make EME persistent license support optional

The recent refactoring from KeySystemsInfo meant we lost the option
to control whether persistent license support is enabled (see linked
bug).  Add the option back into the new system.

BUG=internal b/28844432
==========

[halliwell, 2016-06-02 03:16:24]

halliwell@chromium.org changed reviewers:
+ yucliu@chromium.org

[yucliu1, 2016-06-02 16:31:52]

https://codereview.chromium.org/2032923002/diff/1/chromecast/renderer/key_sys...
File chromecast/renderer/key_systems_cast.cc (right):

https://codereview.chromium.org/2032923002/diff/1/chromecast/renderer/key_sys...
chromecast/renderer/key_systems_cast.cc:87: bool
enable_persistent_license_support) {
Is the flag only for playready?

[halliwell, 2016-06-02 16:39:33]

On 2016/06/02 16:31:52, yucliu1 wrote:
>
https://codereview.chromium.org/2032923002/diff/1/chromecast/renderer/key_sys...
> File chromecast/renderer/key_systems_cast.cc (right):
> 
>
https://codereview.chromium.org/2032923002/diff/1/chromecast/renderer/key_sys...
> chromecast/renderer/key_systems_cast.cc:87: bool
> enable_persistent_license_support) {
> Is the flag only for playready?

Yes, we don't have persistent license support on WV at the moment.

[yucliu1, 2016-06-02 16:47:46]

On 2016/06/02 16:39:33, halliwell wrote:
> On 2016/06/02 16:31:52, yucliu1 wrote:
> >
>
https://codereview.chromium.org/2032923002/diff/1/chromecast/renderer/key_sys...
> > File chromecast/renderer/key_systems_cast.cc (right):
> > 
> >
>
https://codereview.chromium.org/2032923002/diff/1/chromecast/renderer/key_sys...
> > chromecast/renderer/key_systems_cast.cc:87: bool
> > enable_persistent_license_support) {
> > Is the flag only for playready?
> 
> Yes, we don't have persistent license support on WV at the moment.

lgtm

[halliwell, 2016-06-03 21:05:04]

The CQ bit was checked by halliwell@chromium.org

[commit-bot: I haz the power, 2016-06-03 21:05:32]

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

[commit-bot: I haz the power, 2016-06-03 22:41:22]

Description was changed from

==========
[Chromecast] Make EME persistent license support optional

The recent refactoring from KeySystemsInfo meant we lost the option
to control whether persistent license support is enabled (see linked
bug).  Add the option back into the new system.

BUG=internal b/28844432
==========

to

==========
[Chromecast] Make EME persistent license support optional

The recent refactoring from KeySystemsInfo meant we lost the option
to control whether persistent license support is enabled (see linked
bug).  Add the option back into the new system.

BUG=internal b/28844432
==========

[commit-bot: I haz the power, 2016-06-03 22:41:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-03 22:43:07]

Description was changed from

==========
[Chromecast] Make EME persistent license support optional

The recent refactoring from KeySystemsInfo meant we lost the option
to control whether persistent license support is enabled (see linked
bug).  Add the option back into the new system.

BUG=internal b/28844432
==========

to

==========
[Chromecast] Make EME persistent license support optional

The recent refactoring from KeySystemsInfo meant we lost the option
to control whether persistent license support is enabled (see linked
bug).  Add the option back into the new system.

BUG=internal b/28844432

Committed: https://crrev.com/73a640d498ea36d6c60daca10c5a2b7979511660
Cr-Commit-Position: refs/heads/master@{#397827}
==========

[commit-bot: I haz the power, 2016-06-03 22:43:08]

Patchset 1 (id:??) landed as
https://crrev.com/73a640d498ea36d6c60daca10c5a2b7979511660
Cr-Commit-Position: refs/heads/master@{#397827}