Remove PPAPI gyp files.

Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in //ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp (referenced only from trusted). This patch merges them into the cpp directory, which now morphs as required for the build type.

R=bbudge
TBR=bradnelson (chrome/test/data/nacl)

Committed: https://crrev.com/9d6a868b4d36d349aedd875a799a47d296776f84
Cr-Commit-Position: refs/heads/master@{#415183}

[brettw, 2016-08-29 21:23:51]

brettw@chromium.org changed reviewers:
+ bbudge@chromium.org

[brettw, 2016-08-29 21:23:58]

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

[bbudge, 2016-08-29 21:55:04]

LGTM (woohoo!)

[brettw, 2016-08-29 22:01:10]

Description was changed from

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previuosly the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which nwo morphs as required for the build type.
==========

to

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which nwo morphs as required for the build type.
==========

[brettw, 2016-08-29 22:10:29]

fixes

[brettw, 2016-08-29 22:10:29]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-08-29 22:10:30]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org
Link to the patchset: https://codereview.chromium.org/2289983002/#ps40001
(title: "fixes")

[commit-bot: I haz the power, 2016-08-29 22:30:32]

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

[brettw, 2016-08-29 22:56:49]

visibility

[brettw, 2016-08-29 22:56:49]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-08-29 22:56:50]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org
Link to the patchset: https://codereview.chromium.org/2289983002/#ps60001
(title: "visibility")

[brettw, 2016-08-30 00:09:18]

Description was changed from

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which nwo morphs as required for the build type.
==========

to

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which now morphs as required for the build type.
==========

[brettw, 2016-08-30 00:09:18]

brettw@chromium.org changed reviewers:
+ bradnelson@chromium.org

[brettw, 2016-08-30 00:09:36]

Description was changed from

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which now morphs as required for the build type.
==========

to

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which now morphs as required for the build type.

R=bbudge
TBR=bradnelson (chrome/test/data/nacl)
==========

[brettw, 2016-08-30 00:10:17]

TBR bradnelson for chrome/test/data/nacl.

Can we delete or update the OWNERS file for this directory? It has "set
noparent" and lists bradnelson and mseaborn.

[brettw, 2016-08-30 00:10:24]

The CQ bit was unchecked by brettw@chromium.org

[brettw, 2016-08-30 00:10:29]

The CQ bit was checked by brettw@chromium.org

[bradnelson, 2016-08-30 00:36:15]

lgtm

[commit-bot: I haz the power, 2016-08-30 02:11:04]

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-08-30 05:52:22]

Description was changed from

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which now morphs as required for the build type.

R=bbudge
TBR=bradnelson (chrome/test/data/nacl)
==========

to

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which now morphs as required for the build type.

R=bbudge
TBR=bradnelson (chrome/test/data/nacl)
==========

[commit-bot: I haz the power, 2016-08-30 05:52:23]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-30 05:54:39]

Description was changed from

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which now morphs as required for the build type.

R=bbudge
TBR=bradnelson (chrome/test/data/nacl)
==========

to

==========
Remove PPAPI gyp files.

Inlines the sources from ppapi_sources.gypi into the respective BUILD.gn files.

Previously the cpp sources were compiled into two targets, one in
//ppapi:ppapi_cpp_sources (referenced only from nacl), and one in //ppapi/cpp
(referenced only from trusted). This patch merges them into the cpp directory,
which now morphs as required for the build type.

R=bbudge
TBR=bradnelson (chrome/test/data/nacl)

Committed: https://crrev.com/9d6a868b4d36d349aedd875a799a47d296776f84
Cr-Commit-Position: refs/heads/master@{#415183}
==========

[commit-bot: I haz the power, 2016-08-30 05:54:41]

Patchset 4 (id:??) landed as
https://crrev.com/9d6a868b4d36d349aedd875a799a47d296776f84
Cr-Commit-Position: refs/heads/master@{#415183}

[Mark Seaborn, 2016-09-01 22:45:22]

FYI, this appears to break the linux_nacl_sdk_build trybot, which stops the
automated NaCl DEPS rolls from succeeding.

This roll failed just now: https://codereview.chromium.org/2303543004/

The failure was:
Updating projects from gyp files...
gyp:
ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_support_extension.gyp
not found (cwd: /b/build/slave/linux_nacl_sdk_build/build/src)

-- from here:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_nacl_sdk...