Support partial interface for test support idls

Source/bindings/derived_sources.gyp

Index: Source/bindings/derived_sources.gyp
diff --git a/Source/bindings/derived_sources.gyp b/Source/bindings/derived_sources.gyp
index 403811f3c10993d9de661eb386e9febb287d16a8..ffe18d083484cae1799608b4a8c834c254bab8f0 100644
--- a/Source/bindings/derived_sources.gyp
+++ b/Source/bindings/derived_sources.gyp
@@ -53,8 +53,17 @@
     ],
     'webcore_test_support_idl_files': [
         '<@(deprecated_perl_webcore_test_support_idl_files)',
+        '<@(deprecated_perl_generated_webcore_test_support_idl_files)',
         '<@(python_webcore_test_support_idl_files)',
     ],
+    'test_support_perl_and_python_idl_files': [
+        '<@(deprecated_perl_webcore_test_support_idl_files)',
+        '<@(python_webcore_test_support_idl_files)',
+    ],
+    'deprecated_perl_bindings_test_support_idl_files': [
+        '<@(deprecated_perl_webcore_test_support_idl_files)',
+        '<@(deprecated_perl_generated_webcore_test_support_idl_files)',
+    ],
     'compiler_module_files': [
         'scripts/idl_compiler.py',
         '<(DEPTH)/third_party/ply/lex.py',
@@ -151,14 +160,18 @@
         # Write sources into a file, so that the action command line won't
         # exceed OS limits.
         'idl_files_list': '<|(idl_files_list.tmp <@(perl_and_python_idl_files))',
+        'test_support_idl_files_list': '<|(test_support_idl_files_list.tmp <@(test_support_perl_and_python_idl_files))',
       },
       'inputs': [
         'scripts/compute_dependencies.py',
         '<(idl_files_list)',
         '<!@(cat <(idl_files_list))',
+        '<(test_support_idl_files_list)',
+        '<!@(cat <(test_support_idl_files_list))',
        ],
        'outputs': [
          '<(SHARED_INTERMEDIATE_DIR)/blink/InterfaceDependencies.txt',
+         '<(SHARED_INTERMEDIATE_DIR)/blink/TestSupportInterfaceDependencies.txt',
          '<@(generated_global_constructors_idl_files)',
          '<(SHARED_INTERMEDIATE_DIR)/blink/EventInterfaces.in',
        ],
@@ -170,6 +183,10 @@
          '<(idl_files_list)',
          '--interface-dependencies-file',
          '<(SHARED_INTERMEDIATE_DIR)/blink/InterfaceDependencies.txt',
+         '--test-support-idl-files-list',

[do-not-use, 2013/09/09 07:42:09]

Nils or haraken likely know better at this point with all the recent changes.
Would it make sense to call compute_dependencies.py twice? once of regular IDLs
and once for test_support ones?

This way we could reuse the same script arguments and avoid duplication in the
scripts.

[Nils Barth (inactive), 2013/09/10 09:52:10]

Agreed, that's much cleaner (to call twice)!
(Second time don't generate the global constructors IDL files.)

+         '<(test_support_idl_files_list)',
+         '--test-support-interface-dependencies-file',
+         '<(SHARED_INTERMEDIATE_DIR)/blink/TestSupportInterfaceDependencies.txt',
          '--window-constructors-file',
          '<(SHARED_INTERMEDIATE_DIR)/blink/WindowConstructors.idl',
          '--workerglobalscope-constructors-file',
@@ -197,7 +214,7 @@
       ],
       'sources': [
         '<@(deprecated_perl_idl_files)',
-        '<@(deprecated_perl_webcore_test_support_idl_files)',
+        '<@(deprecated_perl_bindings_test_support_idl_files)',
       ],
       'rules': [{
         'rule_name': 'deprecated_perl_binding',
@@ -213,6 +230,7 @@
           # 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)/blink/InterfaceDependencies.txt',
+          '<(SHARED_INTERMEDIATE_DIR)/blink/TestSupportInterfaceDependencies.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.
@@ -258,8 +276,10 @@
           '<@(extra_blink_generator_include_dirs)',
           '--interfaceDependenciesFile',
           '<(SHARED_INTERMEDIATE_DIR)/blink/InterfaceDependencies.txt',
+          '--testSupportInterfaceDependenciesFile',

[do-not-use, 2013/09/09 07:27:10]

It is not clear to me yet why the test support IDLs need to be passed separately
now. This leads to some duplication in the scripts.

[Nils Barth (inactive), 2013/09/10 09:52:10]

That's a good question -- it puzzled me at first too.
The issue is that the dependencies file is also used by
action_derivedsourcesallinone.py
to generate derived sources, so we can't put the test support files in there.

(This could be changed, which is a different and cleaner approach, but a bit
more involved.)

+          '<(SHARED_INTERMEDIATE_DIR)/blink/TestSupportInterfaceDependencies.txt',
           '--additionalIdlFiles',
-          '<(deprecated_perl_webcore_test_support_idl_files)',
+          '<(deprecated_perl_generated_webcore_test_support_idl_files)',
           '<@(preprocessor)',
           '<@(write_file_only_if_changed)',
           '<(RULE_INPUT_PATH)',