Merge "Keep reference to CDM to avoid it's premature destruction"

Merge "Keep reference to CDM to avoid it's premature destruction"

Occassionally the CDM object is destroyed before the pipeline that
uses it is stopped, so have WebMediaPlayerImpl keep a reference to
it to prevent this from happening.

This worked in the past as the pipeline was destroyed from
HTMLMediaElement::stop() while the CDM was destroyed later from
MediaKeys::contextDestroyed(). A recent change merged stop()
and contextDestroyed(), so the CDM can be freed before
WebMediaPlayerImpl is done with it.

BUG=652978, 653001
TEST=EME layout tests pass

Review-Url: https://codereview.chromium.org/2402563002
Cr-Commit-Position: refs/heads/master@{#424046}
(cherry picked from commit e616ee95642ecc0b168a8e807abc86ff3633cda8)

Committed: https://chromium.googlesource.com/chromium/src/+/f5e4c23ccfb23ae1fe6e7aac3aed6b81664f29f6

[jrummell, 2016-10-11 23:32:23]

Description was changed from

==========
Merge "Keep reference to CDM to avoid it's premature destruction"

Occassionally the CDM object is destroyed before the pipeline that
uses it is stopped, so have WebMediaPlayerImpl keep a reference to
it to prevent this from happening.

This worked in the past as the pipeline was destroyed from
HTMLMediaElement::stop() while the CDM was destroyed later from
MediaKeys::contextDestroyed(). A recent change merged stop()
and contextDestroyed(), so the CDM can be freed before
WebMediaPlayerImpl is done with it.

BUG=652978,653001
TEST=EME layout tests pass

Review-Url: https://codereview.chromium.org/2402563002
Cr-Commit-Position: refs/heads/master@{#424046}
(cherry picked from commit e616ee95642ecc0b168a8e807abc86ff3633cda8)
==========

to

==========
Merge "Keep reference to CDM to avoid it's premature destruction"

Occassionally the CDM object is destroyed before the pipeline that
uses it is stopped, so have WebMediaPlayerImpl keep a reference to
it to prevent this from happening.

This worked in the past as the pipeline was destroyed from
HTMLMediaElement::stop() while the CDM was destroyed later from
MediaKeys::contextDestroyed(). A recent change merged stop()
and contextDestroyed(), so the CDM can be freed before
WebMediaPlayerImpl is done with it.

BUG=652978,653001
TEST=EME layout tests pass

Review-Url: https://codereview.chromium.org/2402563002
Cr-Commit-Position: refs/heads/master@{#424046}
(cherry picked from commit e616ee95642ecc0b168a8e807abc86ff3633cda8)

Committed:
https://chromium.googlesource.com/chromium/src/+/f5e4c23ccfb23ae1fe6e7aac3aed...
==========

[jrummell, 2016-10-11 23:32:24]

Committed patchset #1 (id:1) manually as
f5e4c23ccfb23ae1fe6e7aac3aed6b81664f29f6.