Split interfaces_info between core and modules

Split interfaces_info between core and modules

GYP split step 3/4

Last remaining step (shown in FIXMEs) is the global constructors.

Needed to modify the interfaces_info merging function due to
existing of partial interface definitions that cross
components, primarily lots of
partial interface Navigator
in modules.
This is not a layering violation (since partial
interfaces *extend* interfaces), but it causes added
complexity.

One nice side effect:
touching a core IDL file means we now only need to parse all
core files, but not modules files, and conversely for
modules, which makes incremental build a bit faster.

R=haraken
TBR=thakis
BUG=358074
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=174942

[Nils Barth (inactive), 2014-05-28 05:59:36]

Split step 3/4:
GYP files in top-level bindings/ dir almost gone!

[haraken, 2014-05-28 06:14:04]

LGTM

https://codereview.chromium.org/304643003/diff/70001/Source/bindings/modules/...
File Source/bindings/modules/idl.gypi (right):

https://codereview.chromium.org/304643003/diff/70001/Source/bindings/modules/...
Source/bindings/modules/idl.gypi:27: #
'<@(modules_generated_dependency_idl_files)',

Just help me understand: Why is this commented out?

[Nils Barth (inactive), 2014-05-28 06:26:09]

Thanks, reply!

https://codereview.chromium.org/304643003/diff/70001/Source/bindings/modules/...
File Source/bindings/modules/idl.gypi (right):

https://codereview.chromium.org/304643003/diff/70001/Source/bindings/modules/...
Source/bindings/modules/idl.gypi:27: #
'<@(modules_generated_dependency_idl_files)',
On 2014/05/28 06:14:04, haraken wrote:
> Just help me understand: Why is this commented out?

Because I'm about to uncomment it in a followup.
(See the definition below, where there's a FIXME to split global constructor
creation;
I didn't want to duplicate the FIXME.)

Since it's an immediate followup, it seems clearer to have commented out section
paralleling the code in core/.

[haraken, 2014-05-28 06:29:24]

https://codereview.chromium.org/304643003/diff/70001/Source/bindings/modules/...
File Source/bindings/modules/idl.gypi (right):

https://codereview.chromium.org/304643003/diff/70001/Source/bindings/modules/...
Source/bindings/modules/idl.gypi:27: #
'<@(modules_generated_dependency_idl_files)',
On 2014/05/28 06:26:10, Nils Barth wrote:
> On 2014/05/28 06:14:04, haraken wrote:
> > Just help me understand: Why is this commented out?
> 
> Because I'm about to uncomment it in a followup.
> (See the definition below, where there's a FIXME to split global constructor
> creation;
> I didn't want to duplicate the FIXME.)
> 
> Since it's an immediate followup, it seems clearer to have commented out
section
> paralleling the code in core/.

Sure!

[Nils Barth (inactive), 2014-05-28 06:30:30]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-05-28 06:30:42]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nbarth@chromium.org/304643003/70001

[commit-bot: I haz the power, 2014-05-28 07:49:55]

Change committed as 174942