Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(868)

Side by Side Diff: Source/bindings/scripts/compute_interfaces_info.py

Issue 169743005: Faster run-bindings-tests (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: removed obsolete method Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 # 2 #
3 # Copyright (C) 2013 Google Inc. All rights reserved. 3 # Copyright (C) 2013 Google Inc. All rights reserved.
4 # 4 #
5 # Redistribution and use in source and binary forms, with or without 5 # Redistribution and use in source and binary forms, with or without
6 # modification, are permitted provided that the following conditions are 6 # modification, are permitted provided that the following conditions are
7 # met: 7 # met:
8 # 8 #
9 # * Redistributions of source code must retain the above copyright 9 # * Redistributions of source code must retain the above copyright
10 # notice, this list of conditions and the following disclaimer. 10 # notice, this list of conditions and the following disclaimer.
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
261 if implemented_interface_info['include_path']] 261 if implemented_interface_info['include_path']]
262 262
263 interface_info.update({ 263 interface_info.update({
264 'dependencies_full_paths': (partial_interfaces_full_paths + 264 'dependencies_full_paths': (partial_interfaces_full_paths +
265 implemented_interfaces_full_paths), 265 implemented_interfaces_full_paths),
266 'dependencies_include_paths': (partial_interfaces_include_paths + 266 'dependencies_include_paths': (partial_interfaces_include_paths +
267 implemented_interfaces_include_paths) , 267 implemented_interfaces_include_paths) ,
268 }) 268 })
269 269
270 270
271 def compute(idl_files, interfaces_info_file, only_if_changed):
Nils Barth (inactive) 2014/03/14 01:01:37 This won't be necessary (indeed, no changes to thi
terry 2014/03/14 16:12:54 Right with compute_interfaces_info no need for thi
272 compute_interfaces_info(idl_files)
273 write_pickle_file(interfaces_info_file, interfaces_info, only_if_changed)
274
275
271 ################################################################################ 276 ################################################################################
272 277
273 def main(): 278 def main():
274 options, args = parse_options() 279 options, args = parse_options()
275 280
276 # Static IDL files are passed in a file (generated at GYP time), due to OS 281 # Static IDL files are passed in a file (generated at GYP time), due to OS
277 # command line length limits 282 # command line length limits
278 with open(options.idl_files_list) as idl_files_list: 283 with open(options.idl_files_list) as idl_files_list:
279 idl_files = [line.rstrip('\n') for line in idl_files_list] 284 idl_files = [line.rstrip('\n') for line in idl_files_list]
280 # Generated IDL files are passed at the command line, since these are in the 285 # Generated IDL files are passed at the command line, since these are in the
281 # build directory, which is determined at build time, not GYP time, so these 286 # build directory, which is determined at build time, not GYP time, so these
282 # cannot be included in the file listing static files 287 # cannot be included in the file listing static files
283 idl_files.extend(args) 288 idl_files.extend(args)
284 289
285 compute_interfaces_info(idl_files) 290 compute(idl_files, options.interfaces_info_file, options.write_file_only_if_ changed)
286 write_pickle_file(options.interfaces_info_file,
287 interfaces_info,
288 options.write_file_only_if_changed)
289 291
290 292
291 if __name__ == '__main__': 293 if __name__ == '__main__':
292 sys.exit(main()) 294 sys.exit(main())
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698