Fix missing GN dependencies.

Fix missing GN dependencies.

Fixes some cases where a dependency between a target's inputs and outputs exists without an explicit dependency.

json_schema_api.gni was generating wrong names, this was fixed.

mojo_application_package.gni was fixed to generate unique names (just a bug I noticed in passing) and support the testonly flag, which is necessary when I added the correct dependency.

The rest of the cases are just adding deps or making existing deps public so that the ability to depend on the target's outputs is passed to dependents.

BUG=487897
Committed: https://crrev.com/eadea21096be21b0d74c93f5b1a0ae6a9cc057d7
Cr-Commit-Position: refs/heads/master@{#330636}

[brettw, 2015-05-18 23:54:21]

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

[brettw, 2015-05-18 23:54:25]

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

[commit-bot: I haz the power, 2015-05-18 23:54:52]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128163007/20001

[commit-bot: I haz the power, 2015-05-19 00:29:01]

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

[commit-bot: I haz the power, 2015-05-19 00:29:02]

Dry run: 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-05-19 17:50:55]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-05-19 17:52:12]

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

[commit-bot: I haz the power, 2015-05-19 17:52:14]

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

[commit-bot: I haz the power, 2015-05-19 17:52:15]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[brettw, 2015-05-19 19:21:44]

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

[commit-bot: I haz the power, 2015-05-19 19:22:33]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128163007/40001

[commit-bot: I haz the power, 2015-05-19 19:49:36]

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

[commit-bot: I haz the power, 2015-05-19 19:49:37]

Dry run: 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_...)

[Dirk Pranke, 2015-05-19 19:59:22]

lgtm

https://codereview.chromium.org/1128163007/diff/40001/build/secondary/tools/g...
File build/secondary/tools/grit/grit_rule.gni (right):

https://codereview.chromium.org/1128163007/diff/40001/build/secondary/tools/g...
build/secondary/tools/grit/grit_rule.gni:449: # target rather than this library.
This needs to be a public deps so
nits .. what's a "grit script running target"? the target calling the grit rule?

"needs to be a public dep" rather than "public deps"?

[Dirk Pranke, 2015-05-19 20:00:34]

oh, might also be a good idea to link to bug 487897 for context.

[brettw, 2015-05-19 22:02:21]

The CQ bit was checked by brettw@chromium.org

[brettw, 2015-05-19 22:02:21]

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/1128163007/#ps60001
(title: "review comments, component build fix")

[commit-bot: I haz the power, 2015-05-19 22:02:49]

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

[commit-bot: I haz the power, 2015-05-19 22:32:09]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-19 22:32:53]

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

[brettw, 2015-05-19 23:12:01]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1137693006/ by brettw@chromium.org.

The reason for reverting is: Broke Windows:
http://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/1568/....