Make EventInterfaces.in build step not depend on bindings generation

Source/core/core_generated.gyp

 #
 # Copyright (C) 2013 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 14 matching lines @@
 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 #
 
 {
   'includes': [
     'core.gypi',
     '../bindings/bindings.gypi',
+    '../modules/modules.gypi',  # for list of event IDL files

[Nils Barth (inactive), 2014/03/27 06:43:51]

I suspect we don't want core/ to depend on modules/,
but this is making the dependency explicit.
(Currently depends on modules/ through bindings/.)

[haraken, 2014/03/27 07:08:59]

Yeah, I don't think it's a right thing to make core/ depend on modules/. Would
you elaborate on why you need the dependency?

[Nils Barth (inactive), 2014/03/27 07:14:12]

core/ already depends on bindings/ that depends on modules/;
this isn't adding a new dependency, but
just makes a transitive dependency direct.

The depedency is needed b/c the generated file Event.cpp (below) depends on the
event
.idl files in both core/ and modules/.
Perhaps bindings/ and this file (and others?) should be moved
elsewhere.

     '../build/features.gypi',
     '../build/scripts/scripts.gypi',
   ],
 
   'targets': [
     {
       'target_name': 'generated_testing_idls',
       'type': 'none',
       'actions': [
         {
@@ skipping 371 matching lines @@
           ],
           'action': [
             'python',
             '../build/scripts/make_element_type_helpers.py',
             'svg/SVGTagNames.in',
             '--output_dir',
             '<(SHARED_INTERMEDIATE_DIR)/blink',
           ],
         },
         {
+          'action_name': 'EventInterfaces',
+          'variables': {
+            'event_idl_files': [
+              '<@(core_event_idl_files)',
+              '<@(modules_event_idl_files)',
+            ],
+            'event_idl_files_list':
+                '<|(event_idl_files_list.tmp <@(event_idl_files))',
+          },
+          'inputs': [
+            '../bindings/scripts/generate_event_interfaces.py',

[Nils Barth (inactive), 2014/03/27 06:43:51]

Will move to build/scripts in followup.

+            '../bindings/scripts/utilities.py',
+            '<@(event_idl_files)',
+          ],
+          'outputs': [
+            '<(SHARED_INTERMEDIATE_DIR)/blink/EventInterfaces.in',
+          ],
+          'action': [
+            'python',
+            '../bindings/scripts/generate_event_interfaces.py',
+            '--event-idl-files-list',
+            '<@(event_idl_files_list)',
+            '--event-interfaces-file',
+            '<(SHARED_INTERMEDIATE_DIR)/blink/EventInterfaces.in',
+            '<@(write_file_only_if_changed)',
+          ],
+        },
+        {
           'action_name': 'EventFactory',
           'inputs': [
             '<@(make_event_factory_files)',
             '<(SHARED_INTERMEDIATE_DIR)/blink/EventInterfaces.in',
             'events/EventAliases.in',
           ],
           'outputs': [
             '<(SHARED_INTERMEDIATE_DIR)/blink/Event.cpp',
             '<(SHARED_INTERMEDIATE_DIR)/blink/EventHeaders.h',
             '<(SHARED_INTERMEDIATE_DIR)/blink/EventInterfaces.h',
@@ skipping 324 matching lines @@
             '../build/scripts/rule_bison.py',
             '<(RULE_INPUT_PATH)',
             '<(SHARED_INTERMEDIATE_DIR)/blink',
             '<(bison_exe)',
           ],
         },
       ],
     },
   ],
 }