Rationalize the top-level BUILD.gn rules a bit

Rationalize the top-level BUILD.gn rules a bit

Historically for reasons that are no longer relevant //mojo/BUILD.gn's
"mojo" target has been treated as the global metatarget to add new
things to, including targets in other top-level directories like
//services. This rationalizes the dependency structure in the top few
build files to more closely match the directory structure. //BUILD.gn
references metatargets in top level directories and
//{services,mojo,shell}/BUILD.gn handle references within those
//directories. This will hopefully make it clearer where to add new
//targets.

R=viettrungluu@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/25dfab20ede060a13acea2adc34164ab0feb17b1

[jamesr, 2015-08-24 18:05:44]

jamesr@chromium.org changed reviewers:
+ viettrungluu@chromium.org

[viettrungluu, 2015-08-24 21:55:29]

lgtm w/nit

https://codereview.chromium.org/1312763004/diff/1/services/BUILD.gn
File services/BUILD.gn (right):

https://codereview.chromium.org/1312763004/diff/1/services/BUILD.gn#newcode81
services/BUILD.gn:81: if (is_linux) {
nit: blank line above? (or remove the blank line above "if (is_android) {"
below)

[jamesr, 2015-08-24 21:58:11]

https://codereview.chromium.org/1312763004/diff/1/services/BUILD.gn
File services/BUILD.gn (right):

https://codereview.chromium.org/1312763004/diff/1/services/BUILD.gn#newcode81
services/BUILD.gn:81: if (is_linux) {
On 2015/08/24 at 21:55:29, viettrungluu wrote:
> nit: blank line above? (or remove the blank line above "if (is_android) {"
below)

sure. i was expecting 'gn format' to have an opinion but it apparently leaves
this alone

[jamesr, 2015-08-24 22:20:23]

Committed patchset #1 (id:1) manually as
25dfab20ede060a13acea2adc34164ab0feb17b1 (presubmit successful).