Split apart the PPAPI GN targets

Split apart the PPAPI GN targets

This puts BUILD files in the corresponding directories without duplicate names.

Adds a template for ppapi examples and writes the build for one example. I only did one at this point to test the infrastructure without making this patch too large.

NOPRESUBMIT=true

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

[brettw, 2015-02-12 22:42:52]

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

[dmichael (off chromium), 2015-02-12 23:44:33]

lgtm

https://codereview.chromium.org/921953002/diff/20001/chrome/renderer/BUILD.gn
File chrome/renderer/BUILD.gn (right):

https://codereview.chromium.org/921953002/diff/20001/chrome/renderer/BUILD.gn...
chrome/renderer/BUILD.gn:83: "//ppapi/shared_impl",
tiny nit: these were sorted in the prior patch, but here you have "host" 3rd... 
is there a reason, or should it be alphabetical?

https://codereview.chromium.org/921953002/diff/20001/ppapi/BUILD.gn
File ppapi/BUILD.gn (right):

https://codereview.chromium.org/921953002/diff/20001/ppapi/BUILD.gn#newcode20
ppapi/BUILD.gn:20: # TODO(GYP) other targets from ppapi_tests.gyp
Are the only things left the (non-audio) examples? Should the TODO move from
here to ppapi/examples/BUILD.gn?

https://codereview.chromium.org/921953002/diff/20001/ppapi/proxy/BUILD.gn
File ppapi/proxy/BUILD.gn (right):

https://codereview.chromium.org/921953002/diff/20001/ppapi/proxy/BUILD.gn#new...
ppapi/proxy/BUILD.gn:329: }
nit: This might be easier to spot if it was right under the "sources" section

https://codereview.chromium.org/921953002/diff/20001/ppapi/shared_impl/BUILD.gn
File ppapi/shared_impl/BUILD.gn (right):

https://codereview.chromium.org/921953002/diff/20001/ppapi/shared_impl/BUILD....
ppapi/shared_impl/BUILD.gn:144: # 64-bit builds.
The comment could be clearer...  the problem seems to be that this applies for
the 64-bit Win build of Chromium (not all 64-bit builds).

Note in my defense: We had no 64-bit Win build back when we pulled these out for
nacl_win64...

[brettw, 2015-02-13 00:20:43]

New patchsets have been uploaded after l-g-t-m from dmichael@chromium.org

[brettw, 2015-02-13 00:20:58]

https://codereview.chromium.org/921953002/diff/20001/chrome/renderer/BUILD.gn
File chrome/renderer/BUILD.gn (right):

https://codereview.chromium.org/921953002/diff/20001/chrome/renderer/BUILD.gn...
chrome/renderer/BUILD.gn:83: "//ppapi/shared_impl",
On 2015/02/12 23:44:33, dmichael wrote:
> tiny nit: these were sorted in the prior patch, but here you have "host"
3rd... 
> is there a reason, or should it be alphabetical?

Done.

https://codereview.chromium.org/921953002/diff/20001/ppapi/BUILD.gn
File ppapi/BUILD.gn (right):

https://codereview.chromium.org/921953002/diff/20001/ppapi/BUILD.gn#newcode20
ppapi/BUILD.gn:20: # TODO(GYP) other targets from ppapi_tests.gyp
On 2015/02/12 23:44:33, dmichael wrote:
> Are the only things left the (non-audio) examples? Should the TODO move from
> here to ppapi/examples/BUILD.gn?

No, there's everything in the ppapi/tests directory as well.

https://codereview.chromium.org/921953002/diff/20001/ppapi/shared_impl/BUILD.gn
File ppapi/shared_impl/BUILD.gn (right):

https://codereview.chromium.org/921953002/diff/20001/ppapi/shared_impl/BUILD....
ppapi/shared_impl/BUILD.gn:144: # 64-bit builds.
On 2015/02/12 23:44:33, dmichael wrote:
> The comment could be clearer...  the problem seems to be that this applies for
> the 64-bit Win build of Chromium (not all 64-bit builds).
> 
> Note in my defense: We had no 64-bit Win build back when we pulled these out
for
> nacl_win64...

Done.

[brettw, 2015-02-13 21:28:00]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-02-13 21:29:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/921953002/60001

[commit-bot: I haz the power, 2015-02-13 21:35:33]

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

[commit-bot: I haz the power, 2015-02-13 21:35:34]

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

[brettw, 2015-02-13 21:42:05]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-02-13 21:43:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/921953002/100001

[commit-bot: I haz the power, 2015-02-13 21:49:53]

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

[commit-bot: I haz the power, 2015-02-13 21:49:54]

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

[brettw, 2015-02-13 22:00:19]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-02-13 22:00:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/921953002/120001

[commit-bot: I haz the power, 2015-02-13 22:27:43]

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

[commit-bot: I haz the power, 2015-02-13 22:27:45]

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

[brettw, 2015-02-14 00:00:34]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-02-14 00:02:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/921953002/140001

[commit-bot: I haz the power, 2015-02-14 01:00:33]

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

[commit-bot: I haz the power, 2015-02-14 01:00:34]

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

[brettw, 2015-02-14 01:01:43]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-02-14 01:02:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/921953002/140001

[commit-bot: I haz the power, 2015-02-14 01:37:39]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-02-14 01:38:12]

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