Cleanup: Remove ppapi_internal.gyp from gyp_remaining target.

Cleanup: Remove ppapi_internal.gyp from gyp_remaining target.

It is already done.

BUG=461019
R=dpranke@chromium.org
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg

Committed: https://crrev.com/79ef072dd9082a432308e55a4adc93848df3449d
Cr-Commit-Position: refs/heads/master@{#323884}

[tfarina, 2015-04-01 22:33:00]

PTAL!

https://codereview.chromium.org/1054753003/diff/1/build/gn_migration.gypi
File build/gn_migration.gypi (right):

https://codereview.chromium.org/1054753003/diff/1/build/gn_migration.gypi#new...
build/gn_migration.gypi:452:
'../third_party/webrtc/tools/tools.gyp:frame_analyzer',
Sorry, this got reverted today, due to a break on Win GN builder. Henrik will
look at it tomorrow.

[Dirk Pranke, 2015-04-01 22:43:22]

In order to be able to actually remove this, we need expand out that wildcard
list of targets and make sure they're added to gn_all in this file, and then do
the same thing for the ppapi/examples target in BUILD.gn.

[Dirk Pranke, 2015-04-01 22:44:09]

On 2015/04/01 22:43:22, Dirk Pranke wrote:
> In order to be able to actually remove this, we need expand out that wildcard
> list of targets and make sure they're added to gn_all in this file, and then
do
> the same thing for the ppapi/examples target in BUILD.gn.

Whoops, meant to add: do you want to do that? If not, I will (was planning to
make such a change today but am happy to let you do it instead ;).

[tfarina, 2015-04-01 22:56:33]

Something like I did in patch set 2?

[Dirk Pranke, 2015-04-01 23:01:33]

On 2015/04/01 22:56:33, tfarina wrote:
> Something like I did in patch set 2?

No, I meant the ppapi_example_* targets that get pulled in from
ppapi_tests.gypi. We don't need to list the ppapi_shared, etc. since those are
libraries used by other things that are listed.

gn_all should be an explicit list of every root-level executable,
shared_library, loadable_module, etc. that gets built; all the things that don't
depend on other things and aren't groups or empty targets.

[tfarina, 2015-04-01 23:10:12]

Another look?

[Dirk Pranke, 2015-04-02 01:25:25]

lgtm, thanks! That's what I was going for.

[tfarina, 2015-04-02 01:31:09]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-04-02 01:32:42]

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

[commit-bot: I haz the power, 2015-04-02 02:46:40]

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

[commit-bot: I haz the power, 2015-04-02 02:46:41]

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

[tfarina, 2015-04-02 02:47:29]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-04-02 02:48:06]

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

[commit-bot: I haz the power, 2015-04-02 03:46:51]

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

[commit-bot: I haz the power, 2015-04-02 03:46:52]

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

[tfarina, 2015-04-02 18:44:35]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-04-02 18:45:20]

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

[commit-bot: I haz the power, 2015-04-02 19:54:09]

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

[commit-bot: I haz the power, 2015-04-02 19:54:10]

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

[tfarina, 2015-04-02 22:36:14]

The CQ bit was checked by tfarina@chromium.org

[tfarina, 2015-04-02 22:36:15]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/1054753003/#ps80001
(title: "REBASE - why clobber isn't working?")

[commit-bot: I haz the power, 2015-04-03 00:52:44]

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

[tfarina, 2015-04-03 03:54:31]

Dirk, any idea what I'm doing wrong?

Sigh!!

[commit-bot: I haz the power, 2015-04-03 06:03:06]

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

[commit-bot: I haz the power, 2015-04-03 06:03:07]

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

[Dirk Pranke, 2015-04-03 17:30:27]

On 2015/04/03 03:54:31, tfarina wrote:
> Dirk, any idea what I'm doing wrong?

weird, for some reason the builder is trying to build a shared library for
'gn_all'. I'll have to try this locally and see what's going on.

[tfarina, 2015-04-04 18:45:07]

Turns out depending on ppapi_example_skeleton was the problem. That target was
making gn_all into a shared library. Ick!

Pushing to CQ after linux_chromium_clobber_rel_gn went green in patch set 16!
\o/

[tfarina, 2015-04-04 18:45:13]

The CQ bit was checked by tfarina@chromium.org

[tfarina, 2015-04-04 18:45:14]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/1054753003/#ps300001
(title: "everything but skeleton")

[commit-bot: I haz the power, 2015-04-04 18:45:19]

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

[commit-bot: I haz the power, 2015-04-04 20:17:07]

Committed patchset #16 (id:300001)

[commit-bot: I haz the power, 2015-04-04 20:18:00]

Patchset 16 (id:??) landed as
https://crrev.com/79ef072dd9082a432308e55a4adc93848df3449d
Cr-Commit-Position: refs/heads/master@{#323884}