Fix test_importer.py to skip files named reftest.list and add test

third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_importer.py

 # 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
 #    disclaimer.
 # 2. Redistributions in binary form must reproduce the above
@@ skipping 212 matching lines @@
                     if not self.options.dry_run and self.import_in_place:
                         _log.info("  pruning %s" % path_base)
                         self.filesystem.remove(path_full)
                         continue
                     else:
                         continue
                 # FIXME: This block should really be a separate function, but the early-continues make that difficult.
 
                 if filename.startswith('.') or filename.endswith('.pl'):
                     continue  # For some reason the w3c repo contains random perl scripts we don't care about.
-                if filename == 'OWNERS':
+                if filename == 'OWNERS' or filename == 'reftest.list':
                     continue  # These files fail our presubmits.
+                # see  http://crbug.com/584660 and

[Dirk Pranke, 2016/06/07 22:05:53]

I probably wouldn't have added these comments; while I agree with qyearsley@
that sometimes it is helpful to have comments that link to bug numbers, in this
case it's fairly obvious that we're intentionally skipping files with these
names, and you can use version control to figure out the context for why this
was added. Otherwise every line in the file would eventually grow multiple bug
numbers :).

However, if you did want to explain *why* we're skipping them, it's better to do
so directly if you can do so in a fairly terse way, to keep the reader from
having to jump to the bug. 

For the OWNERS files, we should figure out if there is any advantage to
importing them in some other form that doesn't cause our presubmits to fail. 

For the reftest.list, we should figure out if we do want to support them at all,
and if we do, the reference to the bug is good but it should be of the form
'TODO(crbug.com/582838): Make reftest manifests work properly'.

+                # http://crbug.com/582838.
 
 
                 fullpath = self.filesystem.join(root, filename)
 
                 mimetype = mimetypes.guess_type(fullpath)
                 if ('html' not in str(mimetype[0]) and
                         'application/xhtml+xml' not in str(mimetype[0]) and
                         'application/xml' not in str(mimetype[0])):
                     copy_list.append({'src': fullpath, 'dest': filename})
                     continue
@@ skipping 170 matching lines @@
                 _log.info('  %s: %s', prefixed_property, total_prefixed_properties[prefixed_property])
 
     def path_too_long(self, source_path):
         """Checks whether a source path is too long to import.
 
         Args:
           Absolute path of file to be imported.
         """
         path_from_repo_base = os.path.relpath(source_path, self.top_of_repo)
         return len(path_from_repo_base) > MAX_PATH_LENGTH