media/base/cdm_config.h: drop unused #include

media/base/cdm_config.h: drop unused #include

BUG=488634
Committed: https://crrev.com/597d8fd4706761687b44cdb03e76c4fdab6359f6
Cr-Commit-Position: refs/heads/master@{#345451}

[hans, 2015-08-25 17:51:49]

hans@chromium.org changed reviewers:
+ sandersd@chromium.org, thakis@chromium.org

[hans, 2015-08-25 17:51:49]

The Clang problem in crbug.com/488634 is fixed now, so we could undo
https://codereview.chromium.org/1133423003/ that removed the MEDIA_EXPORT from
this class.

But is there a compelling reason to export this class if it's just a simple
struct with no methods? Let me know what you think.

If we're not exporting it, we can drop the #include.

[Nico, 2015-08-25 17:55:02]

lgtm, makes sense to me

[ddorwin, 2015-08-25 17:55:44]

ddorwin@chromium.org changed reviewers:
+ ddorwin@chromium.org, xhwang@chromium.org

[ddorwin, 2015-08-25 17:55:44]

xhwang, does this need to be exported for Mojo or anything like that?

[ddorwin, 2015-08-25 17:55:58]

ddorwin@chromium.org changed reviewers:
- ddorwin@chromium.org

[xhwang, 2015-08-25 18:17:33]

On 2015/08/25 17:55:44, ddorwin wrote:
> xhwang, does this need to be exported for Mojo or anything like that?

We need it in MojoCdmService:

https://code.google.com/p/chromium/codesearch#chromium/src/media/mojo/service...

But as hans@ pointed out, since this doesn't have any methods, we probably don't
need to export it.

[hans, 2015-08-25 18:26:31]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2015-08-25 18:27:23]

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

[commit-bot: I haz the power, 2015-08-25 18:38:02]

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

[commit-bot: I haz the power, 2015-08-25 18:38:02]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[hans, 2015-08-25 18:41:51]

sandersd, xhwang: Can I get an owner's stamp from one of you?

[xhwang, 2015-08-25 18:56:34]

lgtm

[xhwang, 2015-08-25 18:56:38]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2015-08-25 18:57:49]

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

[commit-bot: I haz the power, 2015-08-25 22:09:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-08-25 22:10:17]

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