track count of prefixed properties that are imported

Tools/Scripts/webkitpy/w3c/test_importer.py

 #!/usr/bin/env python
 
 # Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions
 # are met:
 #
 # 1. Redistributions of source code must retain the above
 #    copyright notice, this list of conditions and the following
@@ skipping 247 matching lines @@
                     'reftests': reftests, 'jstests': jstests, 'total_tests': total_tests})
 
     def import_tests(self):
         if self.import_list:
             self.setup_destination_directory()
 
         converter = W3CTestConverter()
         total_imported_tests = 0
         total_imported_reftests = 0
         total_imported_jstests = 0
+        total_prefixed_properties = {}
 
         for dir_to_copy in self.import_list:
 
             total_imported_tests += dir_to_copy['total_tests']
             total_imported_reftests += dir_to_copy['reftests']
             total_imported_jstests += dir_to_copy['jstests']
 
             prefixed_properties = []
 
             if not dir_to_copy['copy_list']:
@@ skipping 37 matching lines @@
                     # FIXME: Maybe doing a file diff is in order here for existing files?
                     # In other words, there's no sense in overwriting identical files, but
                     # there's no harm in copying the identical thing.
                     print 'Importing:', orig_filepath
                     print '       As:', new_filepath
 
                 # Only html, xml, or css should be converted
                 # FIXME: Eventually, so should js when support is added for this type of conversion
                 mimetype = mimetypes.guess_type(orig_filepath)
                 if 'html' in str(mimetype[0]) or 'xml' in str(mimetype[0])  or 'css' in str(mimetype[0]):
-
                     converted_file = converter.convert_for_webkit(new_path, filename=orig_filepath)
 
                     if not converted_file:
                         shutil.copyfile(orig_filepath, new_filepath)  # The file was unmodified.
                     else:
+                        for prefixed_property in converted_file[0]:
+                            total_prefixed_properties.setdefault(prefixed_property, 0)
+                            total_prefixed_properties[prefixed_property] += 1
+
                         prefixed_properties.extend(set(converted_file[0]) - set(prefixed_properties))
                         outfile = open(new_filepath, 'wb')
                         outfile.write(converted_file[1])
                         outfile.close()
                 else:
                     shutil.copyfile(orig_filepath, new_filepath)
 
                 copied_files.append(new_filepath.replace(self._webkit_root, ''))
 
             self.remove_deleted_files(new_path, copied_files)
             self.write_import_log(new_path, copied_files, prefixed_properties)
 
         print 'Import complete'
 
         print 'IMPORTED ' + str(total_imported_tests) + ' TOTAL TESTS'
         print 'Imported ' + str(total_imported_reftests) + ' reftests'
         print 'Imported ' + str(total_imported_jstests) + ' JS tests'
         print 'Imported ' + str(total_imported_tests - total_imported_jstests - total_imported_reftests) + ' pixel/manual tests'
+        print
+        print "Properties needing prefixes (by count):"
+        for prefixed_property in sorted(total_prefixed_properties, key=lambda p: total_prefixed_properties[p]):
+            print "  %s: %s" % (prefixed_property, total_prefixed_properties[prefixed_property])
 
     def setup_destination_directory(self):
         """ Creates a destination directory that mirrors that of the source approved or submitted directory """
 
         self.update_test_status()
 
         start = self.source_directory.find(self.test_status)
         new_subpath = self.source_directory[start:len(self.source_directory)]
 
         destination_directory = os.path.join(self.destination_directory, new_subpath)
@@ skipping 60 matching lines @@
             for prop in prop_list:
                 import_log.write(prop + '\n')
         else:
             import_log.write('None\n')
         import_log.write('------------------------------------------------------------------------\n')
         import_log.write('List of files:\n')
         for item in file_list:
             import_log.write(item + '\n')
 
         import_log.close()