media: Rename MediaKeys to ContentDecryptionModule

media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

TBR=bbudge@chromium.org, dcheng@chromium.org
BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/79b19304442fdea7d869b860095d68a633dfb541
Cr-Commit-Position: refs/heads/master@{#438242}

[xhwang, 2016-12-10 06:11:17]

Description was changed from

==========
media: Rename MediaKeys to ContentDecryptionModule

BUG=309237
TEST=No functionality change. Rename only.
==========

to

==========
media: Rename MediaKeys to ContentDecryptionModule

BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xhwang, 2016-12-10 06:11:41]

The CQ bit was checked by xhwang@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-10 06:11:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-10 06:30:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-10 06:30:11]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[xhwang, 2016-12-12 18:11:49]

The CQ bit was checked by xhwang@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-12 18:12:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-12 18:43:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-12 18:43:45]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[xhwang, 2016-12-12 18:52:21]

The CQ bit was checked by xhwang@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-12 18:53:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xhwang, 2016-12-12 18:55:29]

Description was changed from

==========
media: Rename MediaKeys to ContentDecryptionModule

BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xhwang, 2016-12-12 18:55:58]

xhwang@chromium.org changed reviewers:
+ jrummell@chromium.org

[xhwang, 2016-12-12 19:44:03]

The CQ bit was checked by xhwang@chromium.org to run a CQ dry run

[xhwang, 2016-12-12 19:44:28]

Patchset #3 (id:40001) has been deleted

[xhwang, 2016-12-12 19:44:41]

Patchset #1 (id:1) has been deleted

[xhwang, 2016-12-12 19:44:51]

Patchset #1 (id:20001) has been deleted

[commit-bot: I haz the power, 2016-12-12 19:45:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xhwang, 2016-12-12 19:48:51]

jrummell: Another renaming CL. PTAL

[DaleCurtis, 2016-12-12 20:38:55]

Might try adding an entry to the hyper-blame list so that blame works correctly
in the future.

https://commondatastorage.googleapis.com/chrome-infra-docs/flat/depot_tools/d...

[DaleCurtis, 2016-12-12 20:39:29]

Well, after I said that, despite the large size, I realized you're only actually
renaming 1 file, so nevermind :)

[xhwang, 2016-12-12 20:51:53]

On 2016/12/12 20:39:29, DaleCurtis wrote:
> Well, after I said that, despite the large size, I realized you're only
actually
> renaming 1 file, so nevermind :)

I didn't know hyper-blame until seeing your comment :) So thank you; I learned
something today!

After looking at the existing .git-blame-ignore-revs I agree this change is a
probably too small to be worth adding to the list. So I'll follow the "if in
doubt, do not put it on this list" rule :)

[jrummell, 2016-12-12 22:08:40]

lgtm with lots of nit about including/not including the header.

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
File chromecast/media/cdm/cast_cdm_factory.h (right):

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
chromecast/media/cdm/cast_cdm_factory.h:11: #include "media/base/cdm_factory.h"
nit: Should include content_decryption_module.h since the CB's are used below.

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
File chromecast/media/cdm/cast_cdm_proxy.h (right):

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
chromecast/media/cdm/cast_cdm_proxy.h:11: #include
"chromecast/media/cdm/cast_cdm.h"
nit: Should include content_decryption_module.h since it's used directly in this
header.

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
content/renderer/media/android/webmediaplayer_android.cc:39: #include
"media/base/content_decryption_module.h"
nit: Not sure why this is needed.

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
File content/renderer/media/cdm/ppapi_decryptor.h (right):

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
content/renderer/media/cdm/ppapi_decryptor.h:34: // all calls to the
PluginInstance.
nit: Can you combine the comment lines?

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
File content/renderer/media/cdm/render_cdm_factory.h (right):

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
content/renderer/media/cdm/render_cdm_factory.h:13: #include
"media/base/cdm_factory.h"
nit: I think including content_decryption_module.h is needed as
SessionMessageCB, ... are referenced.

https://codereview.chromium.org/2568463003/diff/60001/media/base/android/andr...
File media/base/android/android_cdm_factory.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/android/andr...
media/base/android/android_cdm_factory.cc:47:
scoped_refptr<ContentDecryptionModule> cdm(
nit: Should include content_decryption_module.h since it's not included and the
matching .h file isn't in this change?

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_callback...
File media/base/cdm_callback_promise.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_callback...
media/base/cdm_callback_promise.h:15: #include
"media/base/content_decryption_module.h"
nit: I don't think this is needed.

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_promise.h
File media/base/cdm_promise.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_promise....
media/base/cdm_promise.h:14: #include "media/base/content_decryption_module.h"
nit: I don't think this is needed.

https://codereview.chromium.org/2568463003/diff/60001/media/base/content_decr...
File media/base/content_decryption_module.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/content_decr...
media/base/content_decryption_module.h:64: // http://crbug.com/309237
nit: You can remove this comment now :)

https://codereview.chromium.org/2568463003/diff/60001/media/blink/webcontentd...
File media/blink/webcontentdecryptionmodulesession_impl.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/blink/webcontentd...
media/blink/webcontentdecryptionmodulesession_impl.h:27: class
ContentDecryptionModule;
nit: Not needed as content_decryption_module.h is included.

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/default_cdm_f...
File media/cdm/default_cdm_factory.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/default_cdm_f...
media/cdm/default_cdm_factory.cc:58: scoped_refptr<ContentDecryptionModule> cdm(
nit: Should include content_decryption_module.h.

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/json_web_key_...
File media/cdm/json_web_key_unittest.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/json_web_key_...
media/cdm/json_web_key_unittest.cc:27: ContentDecryptionModule::SessionType
session_type;
nit: Should include content_decryption_module.h.

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/ppapi/externa...
File media/cdm/ppapi/external_clear_key/clear_key_cdm.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/ppapi/externa...
media/cdm/ppapi/external_clear_key/clear_key_cdm.h:43: //
ContentDecryptionModule implementation.
nit: Can you change this to cdm::ContentDecryptionModule?

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
File media/mojo/clients/mojo_cdm_factory.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
media/mojo/clients/mojo_cdm_factory.cc:51:
scoped_refptr<ContentDecryptionModule> cdm(
nit: Should probably include content_decryption_module.h.

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
File media/mojo/clients/mojo_cdm_unittest.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
media/mojo/clients/mojo_cdm_unittest.cc:81: const
scoped_refptr<ContentDecryptionModule>& cdm,
nit; Should probably include content_decryption_module.h

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/remote_r...
File media/remoting/remote_renderer_impl.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/remote_r...
media/remoting/remote_renderer_impl.cc:16: #include
"media/base/content_decryption_module.h"
nit: Don't think this is needed.

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/rpc/prot...
File media/remoting/rpc/proto_utils.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/rpc/prot...
media/remoting/rpc/proto_utils.h:18: #include
"media/base/content_decryption_module.h"
nit: Don't think this is needed.

[xhwang, 2016-12-12 22:32:51]

Thanks for the comments!

PTAL again

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
File chromecast/media/cdm/cast_cdm_factory.h (right):

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
chromecast/media/cdm/cast_cdm_factory.h:11: #include "media/base/cdm_factory.h"
On 2016/12/12 22:08:39, jrummell wrote:
> nit: Should include content_decryption_module.h since the CB's are used below.

Those are part of media::CdmFactory interface and we already includes
cdm_factory.h here.

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
File chromecast/media/cdm/cast_cdm_proxy.h (right):

https://codereview.chromium.org/2568463003/diff/60001/chromecast/media/cdm/ca...
chromecast/media/cdm/cast_cdm_proxy.h:11: #include
"chromecast/media/cdm/cast_cdm.h"
On 2016/12/12 22:08:39, jrummell wrote:
> nit: Should include content_decryption_module.h since it's used directly in
this
> header.

Done.

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
content/renderer/media/android/webmediaplayer_android.cc:39: #include
"media/base/content_decryption_module.h"
On 2016/12/12 22:08:39, jrummell wrote:
> nit: Not sure why this is needed.

removed

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
File content/renderer/media/cdm/ppapi_decryptor.h (right):

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
content/renderer/media/cdm/ppapi_decryptor.h:34: // all calls to the
PluginInstance.
On 2016/12/12 22:08:39, jrummell wrote:
> nit: Can you combine the comment lines?

Done.

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
File content/renderer/media/cdm/render_cdm_factory.h (right):

https://codereview.chromium.org/2568463003/diff/60001/content/renderer/media/...
content/renderer/media/cdm/render_cdm_factory.h:13: #include
"media/base/cdm_factory.h"
On 2016/12/12 22:08:39, jrummell wrote:
> nit: I think including content_decryption_module.h is needed as
> SessionMessageCB, ... are referenced.

ditto, cdm_factory.h covers it

https://codereview.chromium.org/2568463003/diff/60001/media/base/android/andr...
File media/base/android/android_cdm_factory.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/android/andr...
media/base/android/android_cdm_factory.cc:47:
scoped_refptr<ContentDecryptionModule> cdm(
On 2016/12/12 22:08:39, jrummell wrote:
> nit: Should include content_decryption_module.h since it's not included and
the
> matching .h file isn't in this change?

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_callback...
File media/base/cdm_callback_promise.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_callback...
media/base/cdm_callback_promise.h:15: #include
"media/base/content_decryption_module.h"
On 2016/12/12 22:08:39, jrummell wrote:
> nit: I don't think this is needed.

Added comment why I don't want to touch this now.

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_promise.h
File media/base/cdm_promise.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/cdm_promise....
media/base/cdm_promise.h:14: #include "media/base/content_decryption_module.h"
On 2016/12/12 22:08:39, jrummell wrote:
> nit: I don't think this is needed.

Added comment why I don't want to touch this now.

https://codereview.chromium.org/2568463003/diff/60001/media/base/content_decr...
File media/base/content_decryption_module.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/base/content_decr...
media/base/content_decryption_module.h:64: // http://crbug.com/309237
On 2016/12/12 22:08:40, jrummell wrote:
> nit: You can remove this comment now :)

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/blink/webcontentd...
File media/blink/webcontentdecryptionmodulesession_impl.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/blink/webcontentd...
media/blink/webcontentdecryptionmodulesession_impl.h:27: class
ContentDecryptionModule;
On 2016/12/12 22:08:40, jrummell wrote:
> nit: Not needed as content_decryption_module.h is included.

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/default_cdm_f...
File media/cdm/default_cdm_factory.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/default_cdm_f...
media/cdm/default_cdm_factory.cc:58: scoped_refptr<ContentDecryptionModule> cdm(
On 2016/12/12 22:08:40, jrummell wrote:
> nit: Should include content_decryption_module.h.

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/json_web_key_...
File media/cdm/json_web_key_unittest.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/json_web_key_...
media/cdm/json_web_key_unittest.cc:27: ContentDecryptionModule::SessionType
session_type;
On 2016/12/12 22:08:40, jrummell wrote:
> nit: Should include content_decryption_module.h.

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/ppapi/externa...
File media/cdm/ppapi/external_clear_key/clear_key_cdm.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/cdm/ppapi/externa...
media/cdm/ppapi/external_clear_key/clear_key_cdm.h:43: //
ContentDecryptionModule implementation.
On 2016/12/12 22:08:40, jrummell wrote:
> nit: Can you change this to cdm::ContentDecryptionModule?

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
File media/mojo/clients/mojo_cdm_factory.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
media/mojo/clients/mojo_cdm_factory.cc:51:
scoped_refptr<ContentDecryptionModule> cdm(
On 2016/12/12 22:08:40, jrummell wrote:
> nit: Should probably include content_decryption_module.h.

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
File media/mojo/clients/mojo_cdm_unittest.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/mojo/clients/mojo...
media/mojo/clients/mojo_cdm_unittest.cc:81: const
scoped_refptr<ContentDecryptionModule>& cdm,
On 2016/12/12 22:08:40, jrummell wrote:
> nit; Should probably include content_decryption_module.h

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/remote_r...
File media/remoting/remote_renderer_impl.cc (right):

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/remote_r...
media/remoting/remote_renderer_impl.cc:16: #include
"media/base/content_decryption_module.h"
On 2016/12/12 22:08:40, jrummell wrote:
> nit: Don't think this is needed.

Done.

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/rpc/prot...
File media/remoting/rpc/proto_utils.h (right):

https://codereview.chromium.org/2568463003/diff/60001/media/remoting/rpc/prot...
media/remoting/rpc/proto_utils.h:18: #include
"media/base/content_decryption_module.h"
On 2016/12/12 22:08:40, jrummell wrote:
> nit: Don't think this is needed.

Done.

[xhwang, 2016-12-12 22:33:44]

The CQ bit was checked by xhwang@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-12 22:35:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-13 01:14:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-13 01:14:16]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[xhwang, 2016-12-13 06:06:17]

xhwang@chromium.org changed reviewers:
+ bbudge@chromium.org, dcheng@chromium.org, erickung@chromium.org,
slan@chromium.org

[xhwang, 2016-12-13 06:06:18]

slan: Please OWNERS review chromecast/

erickung: Please OWNERS review media/remoting/

bbudge: You are TBRed on trivial renaming changes in
content/renderer/pepper/content_decryptor_delegate.*

dcheng: You are TBRed on trivial renaming changes in
media/base/ipc/media_param_traits_macros.h and
media/mojo/interfaces/content_decryption_module.mojom

ddorwin: FYI

[xhwang, 2016-12-13 06:06:52]

Description was changed from

==========
media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

TBR=bbudge@chromium.org, dcheng@chromium.org
BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[dcheng, 2016-12-13 06:07:14]

ipc rs lgtm

[slan, 2016-12-13 06:23:12]

On 2016/12/13 06:07:14, dcheng wrote:
> ipc rs lgtm

cast rs lgtm

[bbudge, 2016-12-13 15:10:19]

content/renderer/pepper lgtm

[xhwang, 2016-12-13 17:19:13]

The CQ bit was checked by xhwang@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-13 17:19:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[erickung1, 2016-12-13 18:11:25]

media/remoting LGTM

[xhwang, 2016-12-13 18:19:25]

The CQ bit was unchecked by xhwang@chromium.org

[xhwang, 2016-12-13 18:19:27]

The CQ bit was checked by xhwang@chromium.org

[xhwang, 2016-12-13 18:19:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
jrummell@chromium.org
Link to the patchset: https://codereview.chromium.org/2568463003/#ps80001
(title: "comments addressed")

[commit-bot: I haz the power, 2016-12-13 18:19:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xhwang, 2016-12-13 18:20:00]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-12-13 18:20:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-13 18:52:48]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1481653200143300,
"parent_rev": "3ae33bd4edfbbfd56aa582ee42518940c3bb972c", "commit_rev":
"3b38db7428b1d9ace8344326e1b5966cf03ce252"}

[commit-bot: I haz the power, 2016-12-13 18:53:15]

Description was changed from

==========
media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

TBR=bbudge@chromium.org, dcheng@chromium.org
BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

TBR=bbudge@chromium.org, dcheng@chromium.org
BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2568463003
==========

[commit-bot: I haz the power, 2016-12-13 18:53:17]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2016-12-13 18:56:27]

Description was changed from

==========
media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

TBR=bbudge@chromium.org, dcheng@chromium.org
BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2568463003
==========

to

==========
media: Rename MediaKeys to ContentDecryptionModule

This class is really implementing the Content Decryption Module (CDM) in
the spec (https://w3c.github.io/encrypted-media/#definitions). In the
code, it's implementing WebContentDecryptionModule.

TBR=bbudge@chromium.org, dcheng@chromium.org
BUG=309237
TEST=No functionality change. Rename only.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/79b19304442fdea7d869b860095d68a633dfb541
Cr-Commit-Position: refs/heads/master@{#438242}
==========

[commit-bot: I haz the power, 2016-12-13 18:56:29]

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