IDL build: Split global constructors into core and modules

Source/bindings/modules/generated.gyp

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Generate IDL interfaces info for modules, used to generate bindings
 #
 # Design doc: http://www.chromium.org/developers/design-documents/idl-build
 
 {
   'includes': [
-    '../../build/scripts/scripts.gypi',
-    '../bindings.gypi',
-    '../core/core.gypi',
-    '../scripts/scripts.gypi',
+    # ../.. == Source
+    '../../bindings/bindings.gypi',

[haraken, 2014/06/03 15:31:10]

FIXME: remove this?

+    '../../bindings/core/core.gypi',
+    '../../bindings/scripts/scripts.gypi',
+    '../../build/scripts/scripts.gypi',  # FIXME: Needed for event files, should be in modules, not bindings_modules http://crbug.com/358074
+    '../../modules/modules.gypi',

[haraken, 2014/06/03 15:31:10]

Doesn't this GYP also depend on:

- ../../modules/generated.gypi
- ../../bindings/core/generated.gypi
- ../../bindings/core/idl.gypi
- ../../core/core.gypi
- ../../core/generated.gypi

?

In particular, I don't understand why this GYP doesn't depend on
../../bindings/core/generated.gypi and ../../core/core.gypi. Probably is the
"includes" working transitively via ../../bindings/core/core.gypi?

Even if the "includes" works transitively, we don't want to rely on the
transitiveness to make the dependency clearer, given the complexity of the build
dependency around the IDL compiler.

[Nils Barth (inactive), 2014/06/04 03:36:31]

No.

Variables are *expanded* on use;
includes are only needed if you use a variable.

Consider:
a.gypi:
'variables': {'a': 'A'},

b.gypi:
'includes': ['a.gypi'],
'variables': {'b': '<(a)'},

c.gypi:
'includes': ['b.gypi'],
'variables': {'c': '<(b)'},


c.gypi does not need to include a.gypi,
because it never uses the name 'a'.


Transitive includes *are* possible in GYP;
we avoid them for exactly the reason you mention (they're fragile).
They're possible, and would look like this:

a.gypi:
'variables': {'a': 'A'},

b.gypi:
'includes': ['a.gypi'],

c.gypi:
'includes': ['b.gypi'],
'variables': {'c': '<(a)'},  # ERROR: transitive include

+    'generated.gypi',
     'idl.gypi',
     'modules.gypi',
   ],
 
   'targets': [
 ################################################################################
   {
-    'target_name': 'bindings_modules_generated',
+    'target_name': 'modules_event_generated',
     'type': 'none',
     'actions': [
       {
         'action_name': 'event_interfaces',
         'variables': {
           'event_idl_files': [
             '<@(modules_event_idl_files)',
           ],
           'event_idl_files_list':
               '<|(event_idl_files_list.tmp <@(event_idl_files))',
         },
         'inputs': [
           '<(bindings_scripts_dir)/generate_event_interfaces.py',
           '<(bindings_scripts_dir)/utilities.py',
           '<(event_idl_files_list)',
           '<@(event_idl_files)',
         ],
         'outputs': [
-          '<(blink_output_dir)/EventModulesInterfaces.in',
+          # FIXME: should output to bindings_modules_output_dir  http://crbug.com/358074
+          '<(SHARED_INTERMEDIATE_DIR)/blink/EventModulesInterfaces.in',
         ],
         'action': [
           'python',
           '<(bindings_scripts_dir)/generate_event_interfaces.py',
           '--event-idl-files-list',
           '<(event_idl_files_list)',
           '--event-interfaces-file',
-          '<(blink_output_dir)/EventModulesInterfaces.in',
+          '<(SHARED_INTERMEDIATE_DIR)/blink/EventModulesInterfaces.in',
           '--write-file-only-if-changed',
           '<(write_file_only_if_changed)',
           '--suffix',
           'Modules',
         ],
       },
       {
         'action_name': 'EventModulesFactory',
         'inputs': [
           '<@(make_event_factory_files)',
@@ skipping 61 matching lines @@
         'action': [
           'python',
           '../../build/scripts/make_names.py',
           '../../modules/EventTargetModulesFactory.in',
           '--output_dir',
           '<(SHARED_INTERMEDIATE_DIR)/blink',
         ],
       },
     ],
   },
+################################################################################
+  {
+    'target_name': 'modules_global_constructors_idls',
+    'type': 'none',
+    'dependencies': [
+        # FIXME: should be modules_global_objects http://crbug.com/358074
+        '../generated.gyp:global_objects',
+    ],
+    'actions': [{
+      'action_name': 'generate_modules_global_constructors_idls',
+      'inputs': [
+        '<(bindings_scripts_dir)/generate_global_constructors.py',
+        '<(bindings_scripts_dir)/utilities.py',
+        # Only includes main IDL files (exclude dependencies and testing,
+        # which should not appear on global objects).
+        '<(modules_idl_files_list)',
+        '<@(modules_idl_files)',
+        '<(bindings_output_dir)/GlobalObjects.pickle',
+      ],
+      'outputs': [
+        '<@(modules_global_constructors_generated_idl_files)',
+        '<@(modules_global_constructors_generated_header_files)',
+      ],
+      'action': [
+        'python',
+        '<(bindings_scripts_dir)/generate_global_constructors.py',
+        '--idl-files-list',
+        '<(modules_idl_files_list)',
+        '--global-objects-file',
+        '<(bindings_output_dir)/GlobalObjects.pickle',
+        '--write-file-only-if-changed',
+        '<(write_file_only_if_changed)',
+        '--',
+        'Window',
+        '<(blink_modules_output_dir)/WindowModulesConstructors.idl',
+        'SharedWorkerGlobalScope',
+        '<(blink_modules_output_dir)/SharedWorkerGlobalScopeModulesConstructors.idl',
+        'DedicatedWorkerGlobalScope',
+        '<(blink_modules_output_dir)/DedicatedWorkerGlobalScopeModulesConstructors.idl',
+        'ServiceWorkerGlobalScope',
+        '<(blink_modules_output_dir)/ServiceWorkerGlobalScopeModulesConstructors.idl',
+       ],
+       'message':
+         'Generating IDL files for constructors on global objects from modules',
+      }]
+  },
+################################################################################
   {
     'target_name': 'interfaces_info_individual_modules',
     'type': 'none',
     'dependencies': [
-      # FIXME: should be modules_generated_idls
-      # http://crbug.com/358074
-      '../generated.gyp:generated_idls',
+      'modules_global_constructors_idls',
     ],
     'actions': [{
       'action_name': 'compute_interfaces_info_individual_modules',
       'inputs': [
         '<(bindings_scripts_dir)/compute_interfaces_info_individual.py',
         '<(bindings_scripts_dir)/utilities.py',
         '<(modules_static_idl_files_list)',
         '<@(modules_static_idl_files)',
-        # No generated files currently, will add with constructors
-        # '<@(modules_generated_idl_files)',
+        '<@(modules_generated_idl_files)',
       ],
       'outputs': [
         '<(bindings_modules_output_dir)/InterfacesInfoModulesIndividual.pickle',
       ],
       'action': [
         'python',
         '<(bindings_scripts_dir)/compute_interfaces_info_individual.py',
         '--component-dir',
         'modules',
         '--idl-files-list',
         '<(modules_static_idl_files_list)',
         '--interfaces-info-file',
         '<(bindings_modules_output_dir)/InterfacesInfoModulesIndividual.pickle',
         '--write-file-only-if-changed',
         '<(write_file_only_if_changed)',
-        # No generated files currently, will add with constructors
-        # '--',
+        '--',
         # Generated files must be passed at command line
-        # '<@(modules_generated_idl_files)',
+        '<@(modules_generated_idl_files)',
       ],
       'message': 'Computing global information about individual IDL files',
       }]
   },
 ################################################################################
   {
     'target_name': 'interfaces_info',
     'type': 'none',
     'dependencies': [
         '../core/generated.gyp:interfaces_info_individual_core',
@@ skipping 18 matching lines @@
         '<(bindings_core_output_dir)/InterfacesInfoCoreIndividual.pickle',
         '<(bindings_modules_output_dir)/InterfacesInfoModulesIndividual.pickle',
         '<(bindings_modules_output_dir)/InterfacesInfoModules.pickle',
       ],
       'message': 'Computing overall global information about IDL files',
       }]
   },
 ################################################################################
   ],  # targets
 }