Faster run-bindings-tests

Source/bindings/scripts/compute_interfaces_info.py

 #!/usr/bin/python
 #
 # 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.
@@ skipping 328 matching lines @@
             raise IdlInterfaceFileNotFoundError('Could not find the IDL file where the following implemented interface is defined: %s' % key_name)
 
         interface_info['dependencies_full_paths'] = (
                 partial_interfaces_full_paths +
                 implemented_interfaces_full_paths)
         interface_info['dependencies_include_paths'] = (
                 partial_interfaces_include_paths +
                 implemented_interfaces_include_paths)
 
 
+def clear_globals():
+    interfaces_info.clear()
+    partial_interface_files.clear()
+    parent_interfaces.clear()
+    extended_attributes_by_interface.clear()  # interface name -> extended attributes

[Nils Barth (inactive), 2014/03/03 01:32:48]

Could you remove the comment at the end of the line here?
(It's just needed on the initialization line.)

[terry, 2014/03/13 19:58:18]

Done.

+
+
+def compute(idl_files, interfaces_info_file, interface_dependencies_file,

[Nils Barth (inactive), 2014/03/03 06:45:49]

This function (and clear_globals()) won't be necessary once I move
EventInterfaces.in out of compute_interfaces_info
(and remove the duplicate test)!

+            event_names_file, only_if_changed):
+    clear_globals()
+

[Nils Barth (inactive), 2014/03/03 01:32:48]

Could you remove this blank line, as the function will soon be only 3 lines
long?

[terry, 2014/03/13 19:58:18]

Done.

+    compute_interfaces_info(idl_files)
+    write_pickle_file(interfaces_info_file, interfaces_info, only_if_changed)
+    write_dependencies_file(interface_dependencies_file, only_if_changed)

[Nils Barth (inactive), 2014/03/03 01:32:48]

BTW, I'm removing the interface_dependencies_file right now,
and I'll remove the event_names_file in a day or so,
so this will soon be 3 lines long!

[terry, 2014/03/13 19:58:18]

Done.

+    write_event_names_file(event_names_file, only_if_changed)
+
+
 ################################################################################
 
 def main():
     options, args = parse_options()
 
     # Static IDL files are passed in a file (generated at GYP time), due to OS
     # command line length limits
     with open(options.idl_files_list) as idl_files_list:
         idl_files = [line.rstrip('\n') for line in idl_files_list]
     # Generated IDL files are passed at the command line, since these are in the
     # build directory, which is determined at build time, not GYP time, so these
     # cannot be included in the file listing static files
     idl_files.extend(args)
 
-    only_if_changed = options.write_file_only_if_changed
-    compute_interfaces_info(idl_files)
-    write_pickle_file(options.interfaces_info_file, interfaces_info, only_if_changed)
-    write_dependencies_file(options.interface_dependencies_file, only_if_changed)
-    write_event_names_file(options.event_names_file, only_if_changed)
+    compute(idl_files,
+            options.interfaces_info_file,
+            options.interface_dependencies_file,
+            options.event_names_file,
+            options.write_file_only_if_changed)
 
 
 if __name__ == '__main__':
     sys.exit(main())