Move ENABLE_PEPPER_CDMS to a buildflag header.

Move ENABLE_PEPPER_CDMS to a buildflag header.

Removes the global build flag and #define and converts to a buildflag header in
ppapi/features.

A new target "//third_party/widevine/cdm:headers" was added since
widevine_cdm_version.h was unreferenced in the build. Given that this file now
requires //ppapi/features we really need "gn check" to work on it. I made the
existing users of ":version_h" reference this new target which encompasses the
public headers of the widevine CDM.

TBR=jrummell@chromium.org (original reviewer)
TBR=jschuh@chromium.org (render_messages.h owners)
TBR=ddorwin@chromium.org (widevine owners)

Reland of http://crrev.com/2411433002 with official build fix.

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

[brettw, 2016-10-11 21:46:53]

Description was changed from

==========
Move ENABLE_PEPPER_CDMS to a buildflag header.

Removes the global build flag and #define and converts to a buildflag header in
ppapi/features.

A new target "//third_party/widevine/cdm:headers" was added since
widevine_cdm_version.h was unreferenced in the build. Given that this file now
requires //ppapi/features we really need "gn check" to work on it. I made the
existing users of ":version_h" reference this new target which encompasses the
public headers of the widevine CDM.

TBR=jrummell@chromium.org (original reviewer)
TBR=jschuh@chromium.org (render_messages.h owners)
TBR=ddorwin@chromium.org (widevine owners)

Reland of http://crrev.com/2411433002 with official build fix.
==========

to

==========
Move ENABLE_PEPPER_CDMS to a buildflag header.

Removes the global build flag and #define and converts to a buildflag header in
ppapi/features.

A new target "//third_party/widevine/cdm:headers" was added since
widevine_cdm_version.h was unreferenced in the build. Given that this file now
requires //ppapi/features we really need "gn check" to work on it. I made the
existing users of ":version_h" reference this new target which encompasses the
public headers of the widevine CDM.

TBR=jrummell@chromium.org (original reviewer)
TBR=jschuh@chromium.org (render_messages.h owners)
TBR=ddorwin@chromium.org (widevine owners)

Reland of http://crrev.com/2411433002 with official build fix.
==========

[brettw, 2016-10-11 21:46:54]

brettw@chromium.org changed reviewers:
+ ddorwin@chromium.org, jrummell@chromium.org, jschuh@chromium.org
- ddorwin@chromium.org (widevine owners), jrummell@chromium.org (original
reviewer), jschuh@chromium.org (render_messages.h owners)

[brettw, 2016-10-11 21:47:36]

Fix and format

[brettw, 2016-10-11 21:47:36]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-10-11 21:48:06]

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

[brettw, 2016-10-11 21:48:09]

Reland of previous patch.

[jrummell, 2016-10-11 22:29:13]

lgtm

[commit-bot: I haz the power, 2016-10-12 01:49:59]

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

[commit-bot: I haz the power, 2016-10-12 01:50:01]

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_...)

[brettw, 2016-10-12 03:29:58]

The CQ bit was checked by brettw@chromium.org

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

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-10-12 07:25:37]

Description was changed from

==========
Move ENABLE_PEPPER_CDMS to a buildflag header.

Removes the global build flag and #define and converts to a buildflag header in
ppapi/features.

A new target "//third_party/widevine/cdm:headers" was added since
widevine_cdm_version.h was unreferenced in the build. Given that this file now
requires //ppapi/features we really need "gn check" to work on it. I made the
existing users of ":version_h" reference this new target which encompasses the
public headers of the widevine CDM.

TBR=jrummell@chromium.org (original reviewer)
TBR=jschuh@chromium.org (render_messages.h owners)
TBR=ddorwin@chromium.org (widevine owners)

Reland of http://crrev.com/2411433002 with official build fix.
==========

to

==========
Move ENABLE_PEPPER_CDMS to a buildflag header.

Removes the global build flag and #define and converts to a buildflag header in
ppapi/features.

A new target "//third_party/widevine/cdm:headers" was added since
widevine_cdm_version.h was unreferenced in the build. Given that this file now
requires //ppapi/features we really need "gn check" to work on it. I made the
existing users of ":version_h" reference this new target which encompasses the
public headers of the widevine CDM.

TBR=jrummell@chromium.org (original reviewer)
TBR=jschuh@chromium.org (render_messages.h owners)
TBR=ddorwin@chromium.org (widevine owners)

Reland of http://crrev.com/2411433002 with official build fix.
==========

[commit-bot: I haz the power, 2016-10-12 07:25:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-12 07:27:20]

Description was changed from

==========
Move ENABLE_PEPPER_CDMS to a buildflag header.

Removes the global build flag and #define and converts to a buildflag header in
ppapi/features.

A new target "//third_party/widevine/cdm:headers" was added since
widevine_cdm_version.h was unreferenced in the build. Given that this file now
requires //ppapi/features we really need "gn check" to work on it. I made the
existing users of ":version_h" reference this new target which encompasses the
public headers of the widevine CDM.

TBR=jrummell@chromium.org (original reviewer)
TBR=jschuh@chromium.org (render_messages.h owners)
TBR=ddorwin@chromium.org (widevine owners)

Reland of http://crrev.com/2411433002 with official build fix.
==========

to

==========
Move ENABLE_PEPPER_CDMS to a buildflag header.

Removes the global build flag and #define and converts to a buildflag header in
ppapi/features.

A new target "//third_party/widevine/cdm:headers" was added since
widevine_cdm_version.h was unreferenced in the build. Given that this file now
requires //ppapi/features we really need "gn check" to work on it. I made the
existing users of ":version_h" reference this new target which encompasses the
public headers of the widevine CDM.

TBR=jrummell@chromium.org (original reviewer)
TBR=jschuh@chromium.org (render_messages.h owners)
TBR=ddorwin@chromium.org (widevine owners)

Reland of http://crrev.com/2411433002 with official build fix.

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

[commit-bot: I haz the power, 2016-10-12 07:27:22]

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