Support partial interface for test support idls

Tools/Scripts/webkitpy/bindings/main.py

Index: Tools/Scripts/webkitpy/bindings/main.py
diff --git a/Tools/Scripts/webkitpy/bindings/main.py b/Tools/Scripts/webkitpy/bindings/main.py
index 949a4c952ca90e4dcc97bdeba0add9520b423f58..2793be2805886b472594f1ea856e6bbaa0821403 100644
--- a/Tools/Scripts/webkitpy/bindings/main.py
+++ b/Tools/Scripts/webkitpy/bindings/main.py
@@ -53,6 +53,7 @@ SKIP_PYTHON = set([
 ])
 
 input_directory = os.path.join('bindings', 'tests', 'idls')
+support_input_directory = os.path.join('bindings', 'tests', 'idls', 'testing')

[haraken, 2013/10/10 05:57:51]

Why do we want to allocate a separate directory for testing/? Could you put the
test files to bindings/tests/idls/?

[kihong, 2013/10/12 05:43:12]

If these files are in the bindings/test/idls, they are dealt like main idls.
And This is same environment with testing idls.
They are separated from main idl.(under core/testing directory)

 reference_directory = os.path.join('bindings', 'tests', 'results')
 reference_event_names_filename = os.path.join(reference_directory, 'EventInterfaces.in')
 
@@ -87,6 +88,7 @@ class BindingsTests(object):
         self.test_python = test_python
         self.executive = executive
         _, self.interface_dependencies_filename = provider.newtempfile()
+        _, self.derived_sources_list_filename = provider.newtempfile()
         if reset_results:
             self.event_names_filename = os.path.join(reference_directory, 'EventInterfaces.in')
         else:
@@ -136,13 +138,20 @@ class BindingsTests(object):
         return 0
 
     def generate_interface_dependencies(self):
-        idl_files_list_file, idl_files_list_filename = provider.newtempfile()
+        idl_files_list_file, main_idl_files_list_filename = provider.newtempfile()
         idl_paths = [os.path.join(input_directory, input_file)
                      for input_file in os.listdir(input_directory)
                      if input_file.endswith('.idl')]
         idl_files_list_contents = ''.join(idl_path + '\n'
                                           for idl_path in idl_paths)
         os.write(idl_files_list_file, idl_files_list_contents)
+        support_idl_files_list_file, support_idl_files_list_filename = provider.newtempfile()
+        support_idl_paths = [os.path.join(support_input_directory, input_file)
+                     for input_file in os.listdir(support_input_directory)
+                     if input_file.endswith('.idl')]
+        support_idl_files_list_contents = ''.join(idl_path + '\n'
+                                          for idl_path in support_idl_paths)
+        os.write(support_idl_files_list_file, support_idl_files_list_contents)
 
         # Dummy files, required by compute_dependencies but not checked
         _, window_constructors_file = provider.newtempfile()
@@ -151,8 +160,10 @@ class BindingsTests(object):
         _, dedicatedworkerglobalscope_constructors_file = provider.newtempfile()
         cmd = ['python',
                'bindings/scripts/compute_dependencies.py',
-               '--idl-files-list', idl_files_list_filename,
+               '--main-idl-files-list', main_idl_files_list_filename,
+               '--support-idl-files-list', support_idl_files_list_filename,
                '--interface-dependencies-file', self.interface_dependencies_filename,
+               '--bindings-derived-sources-file', self.derived_sources_list_filename,

[haraken, 2013/10/10 05:57:51]

We might want to rename "derived sources" in the code base to "generated". You
can do it in a follow-up CL.

[Nils Barth (inactive), 2013/10/10 06:02:14]

Nono, these are different.
The IDL compiler generates .h/.cpp files.
The derived sources are files like V8DerivedSources01.cpp
that just include the generated ones, hence "derived".

[haraken, 2013/10/10 06:13:24]

I meant, it would be better to rename V8DerivedSources01.cpp to
V8GeneratedFiles01.cpp. We're mixing "derived sources" and "generate
files/sources/code" in the code base.

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

Ok; admittedly "Derived Sources" is a bit vague.

[kihong, 2013/10/12 05:43:12]

I agree, but IMHO, it's better to change this on the other CL.
There are so many derived_xxx files and variables which are not related to this
patch. 

                '--window-constructors-file', window_constructors_file,
                '--workerglobalscope-constructors-file', workerglobalscope_constructors_file,
                '--sharedworkerglobalscope-constructors-file', sharedworkerglobalscope_constructors_file,
@@ -201,14 +212,14 @@ class BindingsTests(object):
                     for (reference_filename, work_filename) in file_pairs])
 
     def run_tests(self):
-        def generate_and_check_output_pl(idl_filename):
+        def generate_and_check_output_pl(idl_filename, directory):
             # Generate output into the reference directory if resetting
             # results, or a temp directory if not.
             if self.reset_results:
                 work_directory = reference_directory
             else:
                 work_directory = provider.newtempdir()
-            idl_path = os.path.join(input_directory, idl_filename)
+            idl_path = os.path.join(directory, idl_filename)
             if self.generate_from_idl_pl(idl_path, work_directory):
                 return False
             if self.reset_results:
@@ -232,9 +243,10 @@ class BindingsTests(object):
         else:
             passed = self.identical_file(reference_event_names_filename,
                                          self.event_names_filename)
-        passed &= all([generate_and_check_output_pl(input_file)
-                       for input_file in os.listdir(input_directory)
-                       if input_file.endswith('.idl')])
+        for directory in [input_directory, support_input_directory]:
+            passed &= all([generate_and_check_output_pl(input_file, directory)
+                           for input_file in os.listdir(directory)
+                           if input_file.endswith('.idl')])
         print
         if self.test_python:
             print 'Python:'