Encrypted Media: Destroy CDM in ProxyMediaKeys' dtor.

Encrypted Media: Destroy CDM in ProxyMediaKeys' dtor.

- Makes sure we destroy the CDM when ProxyMediaKeys is destroyed.
- Changes MediaKeysHostMsg_CancelAllPendingSessionCreations message to a more general MediaKeysHostMsg_DestroyCdm message and cancels pending session creations as part of OnDestroyCdm().

BUG=347154, 347022
TEST=MediaDrmBridge is properly destroyed with all sessions closed when the MediaPlayer is destroyed.
R=dcheng@chromium.org, ddorwin@chromium.org, qinmin@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=253674

[xhwang, 2014-02-26 18:55:52]

https://codereview.chromium.org/181593004/diff/1/content/browser/media/androi...
File content/browser/media/android/browser_media_player_manager.h (right):

https://codereview.chromium.org/181593004/diff/1/content/browser/media/androi...
content/browser/media/android/browser_media_player_manager.h:140: void
OnInitializeCDM(int media_keys_id,
I'll update CDM to Cdm in a followup cleanup CL.

[xhwang, 2014-02-26 18:58:49]

qinmin/ddorwin: Please review everything.

dcheng: Please OWNERS review media_player_messages_android.h which is a
renaming-only change.

ycheo/igsolla: FYI. This CL fixes issues 347154 and 347022.

kkimlabs: FYI as this changes MediaKeysHostMsg_CancelAllPendingSessionCreations.

damienv: FYI as we are adding another message for EME IPC.

[ddorwin, 2014-02-26 19:09:59]

lgtm with comment

https://codereview.chromium.org/181593004/diff/1/content/browser/media/androi...
File content/browser/media/android/browser_media_player_manager.cc (right):

https://codereview.chromium.org/181593004/diff/1/content/browser/media/androi...
content/browser/media/android/browser_media_player_manager.cc:735:
BrowserContext* context =
We should either comment what's going on here (roughly
OnCancelAllPendingSessionCreations)or break it out into a separate function.

[dcheng, 2014-02-26 19:11:40]

content/common/media/media_player_messages_android.h LGTM

[xhwang, 2014-02-26 19:18:22]

comments addressed

[xhwang, 2014-02-26 19:18:58]

https://codereview.chromium.org/181593004/diff/1/content/browser/media/androi...
File content/browser/media/android/browser_media_player_manager.cc (right):

https://codereview.chromium.org/181593004/diff/1/content/browser/media/androi...
content/browser/media/android/browser_media_player_manager.cc:735:
BrowserContext* context =
On 2014/02/26 19:10:00, ddorwin wrote:
> We should either comment what's going on here (roughly
> OnCancelAllPendingSessionCreations)or break it out into a separate function.

Added CancelAllPendingSessionCreations() function with comments.

[qinmin, 2014-02-26 19:42:57]

lgtm

[xhwang, 2014-02-26 21:11:29]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2014-02-26 21:12:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/181593004/2

[xhwang, 2014-02-27 01:45:01]

Committed patchset #2 manually as r253674 (presubmit successful).