media: Remove Browser CDM implementation

media: Remove Browser CDM implementation

This has been superceded by mojo CDM.

BUG=581746, 570711
TEST=This removes an obsolete code path.

Committed: https://crrev.com/aff81aab2643bc59e41d5db64a8cbe2e7e66070a
Cr-Commit-Position: refs/heads/master@{#422140}

[xhwang, 2016-09-29 23:21:21]

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

[commit-bot: I haz the power, 2016-09-29 23:21:47]

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-09-30 00:03:18]

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

[commit-bot: I haz the power, 2016-09-30 00:03:19]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[xhwang, 2016-09-30 03:13:20]

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

[commit-bot: I haz the power, 2016-09-30 03:13:43]

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-09-30 04:50:39]

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

[commit-bot: I haz the power, 2016-09-30 04:50:40]

Dry run: This issue passed the CQ dry run.

[xhwang, 2016-09-30 04:51:26]

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

[xhwang, 2016-09-30 04:51:27]

PTAL

[xhwang, 2016-09-30 05:55:03]

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

[xhwang, 2016-09-30 05:55:04]

Nick: Please OWNERS review content/* that's not under a media/ subfolder.

[xhwang, 2016-09-30 05:58:52]

dcheng: Please OWNERS review code removal in
content/common/media/media_player_messages_android.h.

dpranke: Please OWNERS review code removal in build/config/*

[xhwang, 2016-09-30 05:59:27]

xhwang@chromium.org changed reviewers:
+ dcheng@chromium.org, dpranke@chromium.org

[xhwang, 2016-09-30 05:59:28]

dcheng: Please OWNERS review code removal in
content/common/media/media_player_messages_android.h.

dpranke: Please OWNERS review code removal in build/config/*

[dcheng, 2016-09-30 06:38:21]

ipc lgtm

[Dirk Pranke, 2016-09-30 16:24:11]

lgtm

[DaleCurtis, 2016-09-30 16:42:29]

media/ lgtm \o/

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

https://codereview.chromium.org/2380743004/diff/20001/content/renderer/media/...
content/renderer/media/cdm/render_cdm_factory.h:36: #endif  //
defined(ENABLE_PEPPER_CDMS)
Can this one go away soon too? :)

[ncarter (slow), 2016-09-30 16:52:47]

lgtm

when deleting code, remember to also comb the bug tracker for obsolete issues
that can be closed.

[xhwang, 2016-09-30 17:06:19]

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

https://codereview.chromium.org/2380743004/diff/20001/content/renderer/media/...
content/renderer/media/cdm/render_cdm_factory.h:36: #endif  //
defined(ENABLE_PEPPER_CDMS)
On 2016/09/30 16:42:29, DaleCurtis_OOO_Until_Oct_18 wrote:
> Can this one go away soon too? :)

Depending on how you define "soon" :) Yes, this is what we are working on.

[xhwang, 2016-09-30 17:16:34]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-09-30 17:17:03]

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

[xhwang, 2016-09-30 17:17:49]

slan: FYI

[commit-bot: I haz the power, 2016-09-30 17:23:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-30 17:25:58]

Description was changed from

==========
media: Remove Browser CDM implementation

This has been superceded by mojo CDM.

BUG=581746,570711
TEST=This removes an obsolete code path.
==========

to

==========
media: Remove Browser CDM implementation

This has been superceded by mojo CDM.

BUG=581746,570711
TEST=This removes an obsolete code path.

Committed: https://crrev.com/aff81aab2643bc59e41d5db64a8cbe2e7e66070a
Cr-Commit-Position: refs/heads/master@{#422140}
==========

[commit-bot: I haz the power, 2016-09-30 17:26:00]

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