W3C test importer: only consider tests to be renamed with 100% similarity.

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

Index: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
index 9726fa200c9c79c53be871e5a27ac52b9662bb1a..4f16805503bd56a142791262a4ed9ab2077a4985 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
@@ -424,7 +424,7 @@ class DepsUpdater(object):
 
     def _list_deleted_tests(self):
         """Returns a list of layout tests that have been deleted."""
-        out = self.check_run(['git', 'diff', 'origin/master', '--diff-filter=D', '--name-only'])
+        out = self.check_run(['git', 'diff', 'origin/master', '-M100%', '--diff-filter=D', '--name-only'])

[Dirk Pranke, 2016/10/26 17:50:25]

You might want --no-rename instead, I'm not sure if there'll be much of a
difference.

[qyearsley, 2016/10/26 18:00:19]

It looks like the difference is: with --no-renames, even renames with 100%
similarity are considered a delete and add instead of a rename. With -M100%,
simple renames with no change in content are still considered renames, and not
deletions/additions.

Potentially, I could change this to only remove deleted/renamed files from
TestExpectation, and not update them when they're renamed them, and that would
be a little simpler. But, I think it would still be good to keep the behavior of
updating instead of removing expectation lines when there's just a simple rename
with no change in test content.

         deleted_tests = []
         for line in out.splitlines():
             test = self.finder.layout_test_name(line)
@@ -434,7 +434,7 @@ class DepsUpdater(object):
 
     def _list_renamed_tests(self):
         """Returns a dict mapping source to dest name for layout tests that have been renamed."""
-        out = self.check_run(['git', 'diff', 'origin/master', '--diff-filter=R', '--name-status'])
+        out = self.check_run(['git', 'diff', 'origin/master', '-M100%', '--diff-filter=R', '--name-status'])
         renamed_tests = {}
         for line in out.splitlines():
             _, source_path, dest_path = line.split()