Refactor bindings generation in GYP

Refactor bindings generation in GYP

Bindings generation needs to be separated from compilation to work with NaCl.
Interface generation in mojo_public.gyp was ported to use the new pattern to
ensure the code is live.

BUG=https://code.google.com/p/chromium/issues/detail?id=401761
R=jamesr@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/12a220862704551c94aec4516bd62af71b4d3d6e

[Nick Bray (chromium), 2014-10-28 00:04:38]

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

[jamesr, 2014-10-28 00:54:21]

https://codereview.chromium.org/665083008/diff/1/mojo/public/mojo_public.gyp
File mojo/public/mojo_public.gyp (right):

https://codereview.chromium.org/665083008/diff/1/mojo/public/mojo_public.gyp#...
mojo/public/mojo_public.gyp:198: 'includes': [
'tools/bindings/mojom_bindings_generator_explicit.gypi' ],
does this script exist? where will it come from? i don't see it in my tree

https://codereview.chromium.org/665083008/diff/1/mojo/public/tools/bindings/m...
File mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi (right):

https://codereview.chromium.org/665083008/diff/1/mojo/public/tools/bindings/m...
mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi:96: # TODO can
we nessle direct_dependent_settings inside of direct_dependent_settings?
you mean nestle? or nest? 'nessle' isn't a word afaik

can you please attach a name or a bug number to this TODO?

https://codereview.chromium.org/665083008/diff/1/mojo/public/tools/bindings/m...
mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi:97:
'all_dependent_settings': {
yuck, we don't want *everything* downstream to have these things on the include
path

[Nick Bray (chromium), 2014-10-28 20:57:24]

https://codereview.chromium.org/665083008/diff/1/mojo/public/mojo_public.gyp
File mojo/public/mojo_public.gyp (right):

https://codereview.chromium.org/665083008/diff/1/mojo/public/mojo_public.gyp#...
mojo/public/mojo_public.gyp:198: 'includes': [
'tools/bindings/mojom_bindings_generator_explicit.gypi' ],
On 2014/10/28 00:54:21, jamesr wrote:
> does this script exist? where will it come from? i don't see it in my tree

It's a new file, but git's heuristics cause it to diff new files against similar
existing files, when they're close enough.  In this case it chose
mojom_bindings_generator.gypi, which this file was derived from.  Net effect:
the diff is useful (at least this time), but confusing.

https://codereview.chromium.org/665083008/diff/1/mojo/public/tools/bindings/m...
File mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi (right):

https://codereview.chromium.org/665083008/diff/1/mojo/public/tools/bindings/m...
mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi:96: # TODO can
we nessle direct_dependent_settings inside of direct_dependent_settings?
On 2014/10/28 00:54:21, jamesr wrote:
> you mean nestle? or nest? 'nessle' isn't a word afaik
> 
> can you please attach a name or a bug number to this TODO?

Done.

https://codereview.chromium.org/665083008/diff/1/mojo/public/tools/bindings/m...
mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi:97:
'all_dependent_settings': {
On 2014/10/28 00:54:21, jamesr wrote:
> yuck, we don't want *everything* downstream to have these things on the
include
> path

OK, spent the time to figure out if direct_dependent_settings can be set inside
direct_dependent_settings.  It can.  Uhg/w00t.

[jamesr, 2014-10-29 00:28:33]

cool, I understand now.  lgtm

[Nick Bray (chromium), 2014-10-29 18:15:23]

Committed patchset #4 (id:60001) manually as
12a220862704551c94aec4516bd62af71b4d3d6e (presubmit successful).

[bradn, 2014-11-11 23:48:22]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[bradn, 2014-11-11 23:48:23]

lgtm

https://codereview.chromium.org/665083008/diff/60001/mojo/public/tools/bindin...
File mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi (right):

https://codereview.chromium.org/665083008/diff/60001/mojo/public/tools/bindin...
mojo/public/tools/bindings/mojom_bindings_generator_explicit.gypi:111: # Include
paths needed to find the generated header files and their transitive
>80