Added PRESUBMIT.py for binary_size tools.

tools/binary_size/explain_binary_size_delta.py

 #!/usr/bin/env python
 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Describe the size difference of two binaries.
 
 Generates a description of the size difference of two binaries based
 on the difference of the size of various symbols.
 
@@ skipping 89 matching lines @@
 
   # We have now analyzed all symbols that are in cache1 and removed all of
   # the encountered symbols from cache2. What's left in cache2 is the new
   # symbols.
   for key, bucket2 in cache2.iteritems():
     for symbol_name, symbol_size in bucket2.items():
       added.append((key[0], key[1], symbol_name, None, symbol_size))
   return (added, removed, changed, unchanged)
 
 
-def TestCompare():
-  # List entries have form: symbol_name, symbol_type, symbol_size, file_path
-  symbol_list1 = (
-    # File with one symbol, left as-is.
-    ( 'unchanged', 't', 1000, '/file_unchanged' ),
-    # File with one symbol, changed.
-    ( 'changed', 't', 1000, '/file_all_changed' ),
-    # File with one symbol, deleted.
-    ( 'removed', 't', 1000, '/file_all_deleted' ),
-    # File with two symbols, one unchanged, one changed, same bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_changed' ),
-    ( 'changed', 't', 1000, '/file_pair_unchanged_changed' ),
-    # File with two symbols, one unchanged, one deleted, same bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_removed' ),
-    ( 'removed', 't', 1000, '/file_pair_unchanged_removed' ),
-    # File with two symbols, one unchanged, one added, same bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_added' ),
-    # File with two symbols, one unchanged, one changed, different bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_diffbuck_changed' ),
-    ( 'changed', '@', 1000, '/file_pair_unchanged_diffbuck_changed' ),
-    # File with two symbols, one unchanged, one deleted, different bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_diffbuck_removed' ),
-    ( 'removed', '@', 1000, '/file_pair_unchanged_diffbuck_removed' ),
-    # File with two symbols, one unchanged, one added, different bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_diffbuck_added' ),
-    # File with four symbols, one added, one removed, one changed, one unchanged
-    ( 'size_changed', 't', 1000, '/file_tetra' ),
-    ( 'removed', 't', 1000, '/file_tetra' ),
-    ( 'unchanged', 't', 1000, '/file_tetra' ),
-  )
-
-  symbol_list2 = (
-    # File with one symbol, left as-is.
-    ( 'unchanged', 't', 1000, '/file_unchanged' ),
-    # File with one symbol, changed.
-    ( 'changed', 't', 2000, '/file_all_changed' ),
-    # File with two symbols, one unchanged, one changed, same bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_changed' ),
-    ( 'changed', 't', 2000, '/file_pair_unchanged_changed' ),
-    # File with two symbols, one unchanged, one deleted, same bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_removed' ),
-    # File with two symbols, one unchanged, one added, same bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_added' ),
-    ( 'added', 't', 1000, '/file_pair_unchanged_added' ),
-    # File with two symbols, one unchanged, one changed, different bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_diffbuck_changed' ),
-    ( 'changed', '@', 2000, '/file_pair_unchanged_diffbuck_changed' ),
-    # File with two symbols, one unchanged, one deleted, different bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_diffbuck_removed' ),
-    # File with two symbols, one unchanged, one added, different bucket
-    ( 'unchanged', 't', 1000, '/file_pair_unchanged_diffbuck_added' ),
-    ( 'added', '@', 1000, '/file_pair_unchanged_diffbuck_added' ),
-    # File with four symbols, one added, one removed, one changed, one unchanged
-    ( 'size_changed', 't', 2000, '/file_tetra' ),
-    ( 'unchanged', 't', 1000, '/file_tetra' ),
-    ( 'added', 't', 1000, '/file_tetra' ),
-    # New file with one symbol added
-    ( 'added', 't', 1000, '/file_new' ),
-  )
-
-  # Here we go
-  (added, removed, changed, unchanged) = Compare(symbol_list1, symbol_list2)
-
-  # File with one symbol, left as-is.
-  assert ('/file_unchanged', 't', 'unchanged', 1000, 1000) in unchanged
-  # File with one symbol, changed.
-  assert ('/file_all_changed', 't', 'changed', 1000, 2000) in changed
-  # File with one symbol, deleted.
-  assert ('/file_all_deleted', 't', 'removed', 1000, None) in removed
-  # New file with one symbol added
-  assert ('/file_new', 't', 'added', None, 1000) in added
-  # File with two symbols, one unchanged, one changed, same bucket
-  assert ('/file_pair_unchanged_changed',
-          't', 'unchanged', 1000, 1000) in unchanged
-  assert ('/file_pair_unchanged_changed',
-          't', 'changed', 1000, 2000) in changed
-  # File with two symbols, one unchanged, one removed, same bucket
-  assert ('/file_pair_unchanged_removed',
-          't', 'unchanged', 1000, 1000) in unchanged
-  assert ('/file_pair_unchanged_removed',
-          't', 'removed', 1000, None) in removed
-  # File with two symbols, one unchanged, one added, same bucket
-  assert ('/file_pair_unchanged_added',
-          't', 'unchanged', 1000, 1000) in unchanged
-  assert ('/file_pair_unchanged_added',
-          't', 'added', None, 1000) in added
-  # File with two symbols, one unchanged, one changed, different bucket
-  assert ('/file_pair_unchanged_diffbuck_changed',
-          't', 'unchanged', 1000, 1000) in unchanged
-  assert ('/file_pair_unchanged_diffbuck_changed',
-          '@', 'changed', 1000, 2000) in changed
-  # File with two symbols, one unchanged, one removed, different bucket
-  assert ('/file_pair_unchanged_diffbuck_removed',
-          't', 'unchanged', 1000, 1000) in unchanged
-  assert ('/file_pair_unchanged_diffbuck_removed',
-          '@', 'removed', 1000, None) in removed
-  # File with two symbols, one unchanged, one added, different bucket
-  assert ('/file_pair_unchanged_diffbuck_added',
-          't', 'unchanged', 1000, 1000) in unchanged
-  assert ('/file_pair_unchanged_diffbuck_added',
-          '@', 'added', None, 1000) in added
-  # File with four symbols, one added, one removed, one changed, one unchanged
-  assert ('/file_tetra', 't', 'size_changed', 1000, 2000) in changed
-  assert ('/file_tetra', 't', 'unchanged', 1000, 1000) in unchanged
-  assert ('/file_tetra', 't', 'added', None, 1000) in added
-  assert ('/file_tetra', 't', 'removed', 1000, None) in removed
-
-  # Now check final stats.
-  CrunchStats(added, removed, changed, unchanged, True, True)
-
-
 def CrunchStats(added, removed, changed, unchanged, showsources, showsymbols):
   """Outputs to stdout a summary of changes based on the symbol lists."""
   print 'Symbol statistics:'
   sources_with_new_symbols = set()
   new_symbols_size = 0
   new_symbols_by_path = {}
   for file_path, symbol_type, symbol_name, size1, size2 in added:
     sources_with_new_symbols.add(file_path)
     new_symbols_size += size2
     bucket = new_symbols_by_path.setdefault(file_path, [])
@@ skipping 46 matching lines @@
   removed_sources = (sources_with_removed_symbols -
     maybe_unchanged_sources -
     sources_with_new_symbols)
   partially_changed_sources = (sources_with_changed_symbols |
     sources_with_new_symbols |
     sources_with_removed_symbols) - removed_sources - new_sources
   allFiles = (sources_with_new_symbols |
     sources_with_removed_symbols |
     sources_with_changed_symbols |
     maybe_unchanged_sources)
-  print 'Source stats: '
+  print 'Source stats:'
   print('  %d sources encountered.' % len(allFiles))
   print('  %d completely new.' % len(new_sources))
   print('  %d removed completely.' % len(removed_sources))
   print('  %d partially changed.' % len(partially_changed_sources))
   print('  %d completely unchanged.' % len(unchanged_sources))
   remainder = (allFiles - new_sources - removed_sources -
     partially_changed_sources - unchanged_sources)
   assert len(remainder) == 0
 
   if not showsources:
@@ skipping 74 matching lines @@
   parser.add_option('--nm1', metavar='PATH',
                     help='the nm dump of the first library')
   parser.add_option('--nm2', metavar='PATH',
                     help='the nm dump of the second library')
   parser.add_option('--showsources', action='store_true', default=False,
                     help='show per-source statistics')
   parser.add_option('--showsymbols', action='store_true', default=False,
                     help='show all symbol information; implies --showfiles')
   parser.add_option('--verbose', action='store_true', default=False,
                     help='output internal debugging stuff')
-  parser.add_option('--selftest', action='store_true', default=False,
-                    help='run internal diagnosis')
   opts, _args = parser.parse_args()
 
-  if opts.selftest:
-    TestCompare()
-    return
-
   if not opts.nm1:
     parser.error('--nm1 is required')
   if not opts.nm2:
     parser.error('--nm2 is required')
   symbols = []
   for path in [opts.nm1, opts.nm2]:
     with file(path, 'r') as nm_input:
       if opts.verbose:
         print 'parsing ' + path + '...'
       symbols.append(list(binary_size_utils.ParseNm(nm_input)))
   (added, removed, changed, unchanged) = Compare(symbols[0], symbols[1])
   CrunchStats(added, removed, changed, unchanged,
     opts.showsources | opts.showsymbols, opts.showsymbols)
 
 if __name__ == '__main__':
   sys.exit(main())