Rename deprecated "supplemental_dependencies" and clean up IDL dependency script

Source/bindings/derived_sources.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 82 matching lines @@
     ],
   },
 
   'target_defaults': {
     'variables': {
       'optimize': 'max',
     },
   },
 
   'targets': [{
-    'target_name': 'supplemental_dependencies',
+    'target_name': 'interface_dependencies',
     'type': 'none',
     'actions': [{
-      'action_name': 'generatePartialInterfacesDependency',
+      'action_name': 'compute_interface_dependencies',

[Nils Barth (inactive), 2013/07/09 05:04:32]

(Also make naming convention consistent: remove only example of camelCase.)

       'variables': {
         # Write sources into a file, so that the action command line won't
         # exceed OS limits.
         'idl_files_list': '<|(idl_files_list.tmp <@(idl_files))',
       },
       'inputs': [
-        'scripts/preprocess_idls.py',
+        'scripts/compute_dependencies.py',
         '<(idl_files_list)',
         '<!@(cat <(idl_files_list))',
        ],
        'outputs': [
-         '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+         '<(SHARED_INTERMEDIATE_DIR)/interface_dependencies.txt',

[Nils Barth (inactive), 2013/07/09 05:04:32]

.txt b/c text file (indeed Space Separated Values); better extension?

          '<@(generated_global_constructors_idl_files)',
          '<(SHARED_INTERMEDIATE_DIR)/EventNames.in',
        ],
        'msvs_cygwin_shell': 0,
        'action': [
          'python',
-         'scripts/preprocess_idls.py',
+         'scripts/compute_dependencies.py',
          '--idl-files-list',
          '<(idl_files_list)',
-         '--supplemental-dependency-file',
-         '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+         '--interface-dependencies-file',
+         '<(SHARED_INTERMEDIATE_DIR)/interface_dependencies.txt',

[haraken, 2013/07/09 05:21:38]

Nit: InterfaceDependencies.txt

[Nils Barth (inactive), 2013/07/09 06:00:40]

Got it:
CamelCase.idl  <<  data
lowercase_with_underscores.py  <<  code

          '--window-constructors-file',
          '<(SHARED_INTERMEDIATE_DIR)/WindowConstructors.idl',
          '--workerglobalscope-constructors-file',
          '<(SHARED_INTERMEDIATE_DIR)/WorkerGlobalScopeConstructors.idl',
          '--sharedworkerglobalscope-constructors-file',
          '<(SHARED_INTERMEDIATE_DIR)/SharedWorkerGlobalScopeConstructors.idl',
          '--dedicatedworkerglobalscope-constructors-file',
          '<(SHARED_INTERMEDIATE_DIR)/DedicatedWorkerGlobalScopeConstructors.idl',
          '--event-names-file',
          '<(SHARED_INTERMEDIATE_DIR)/EventNames.in',
          '<@(write_file_only_if_changed)',
        ],
        'message': 'Resolving partial interfaces dependencies in all IDL files',
       }]
     },
     {
       'target_name': 'bindings_sources',
       'type': 'none',
       # The 'binding' rule generates .h files, so mark as hard_dependency, per:
       # https://code.google.com/p/gyp/wiki/InputFormatReference#Linking_Dependencies
       'hard_dependency': 1,
       'dependencies': [
-        'supplemental_dependencies',
+        'interface_dependencies',
         '../core/core_derived_sources.gyp:generate_test_support_idls',
       ],
       'sources': [
         '<@(idl_files)',
         '<@(webcore_test_support_idl_files)',
       ],
       'rules': [{
         'rule_name': 'binding',
         'extension': 'idl',
         'msvs_external_rule': 1,
         'inputs': [
           'scripts/generate-bindings.pl',
           'scripts/CodeGeneratorV8.pm',
           'scripts/IDLParser.pm',
           'scripts/IDLSerializer.pm',
           '../core/scripts/preprocessor.pm',
           'scripts/IDLAttributes.txt',
           # FIXME: If the dependency structure changes, we rebuild all files,
           # since we're not computing dependencies file-by-file in the build.
-          '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+          '<(SHARED_INTERMEDIATE_DIR)/interface_dependencies.txt',
           # FIXME: Similarly, if any partial interface changes, rebuild
           # everything, since every IDL potentially depends on them, because
           # we're not computing dependencies file-by-file.
-          '<!@pymod_do_main(supplemental_idl_files <@(idl_files))',
+          #
+          # If a new partial interface is added, need to regyp to update these
+          # dependencies, as these are computed statically at gyp runtime.
+          '<!@pymod_do_main(list_idl_files_with_partial_interface <@(idl_files))',
           # Generated IDLs are all partial interfaces, hence everything
           # potentially depends on them.
           '<@(generated_global_constructors_idl_files)',
         ],
         'outputs': [
           '<(SHARED_INTERMEDIATE_DIR)/webkit/bindings/V8<(RULE_INPUT_ROOT).cpp',
           '<(SHARED_INTERMEDIATE_DIR)/webkit/bindings/V8<(RULE_INPUT_ROOT).h',
         ],
         'variables': {
           # IDL include paths. The generator will search recursively for IDL
@@ skipping 18 matching lines @@
           '-I../../../JSON/out/lib/perl5',
           'scripts/generate-bindings.pl',
           '--outputDir',
           '<(SHARED_INTERMEDIATE_DIR)/webkit/bindings',
           '--idlAttributesFile',
           'scripts/IDLAttributes.txt',
           '--defines',
           '<(feature_defines)',
           '<@(generator_include_dirs)',
           '<@(extra_blink_generator_include_dirs)',
-          '--supplementalDependencyFile',
-          '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+          '--interfaceDependenciesFile',
+          '<(SHARED_INTERMEDIATE_DIR)/interface_dependencies.txt',
           '--additionalIdlFiles',
           '<(webcore_test_support_idl_files)',
           '<@(preprocessor)',
           '<@(write_file_only_if_changed)',
           '<(RULE_INPUT_PATH)',
         ],
         'message': 'Generating binding from <(RULE_INPUT_PATH)',
       }],
     },
     {
       'target_name': 'bindings_derived_sources',
       'type': 'none',
       'dependencies': [
-        'supplemental_dependencies',
+        'interface_dependencies',
         'bindings_sources',
       ],
       'actions': [{
         'action_name': 'derived_sources_all_in_one',
         'inputs': [
           '../core/scripts/action_derivedsourcesallinone.py',
-          '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+          '<(SHARED_INTERMEDIATE_DIR)/interface_dependencies.txt',
         ],
         'outputs': [
           '<@(derived_sources_aggregate_files)',
         ],
         'action': [
           'python',
           '../core/scripts/action_derivedsourcesallinone.py',
-          '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+          '<(SHARED_INTERMEDIATE_DIR)/interface_dependencies.txt',
           '--',
           '<@(derived_sources_aggregate_files)',
         ],
         'message': 'Generating bindings derived sources',
       }],
     },
   ],
 }