Use Port.skips_test in copy-existing-baselines command.

third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py

Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
index db5290c411699767897d69c4115867e352f06667..3b80975914069e15ffdac1be71ddd0a6e5602036 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
@@ -930,7 +930,7 @@ class Port(object):
         """Checks whether the given test is skipped for this port.
 
         This should return True if the test is skipped because the port
-        runs smoke tests only, or because there's a skip test expectation line.
+        runs smoke tests only, or because the

[chenwilliam, 2016/10/31 22:49:29]

I think this patch truncated the comment on accident (I just noticed it while
working on this file).

[qyearsley, 2016/10/31 23:05:21]

Good catch, thanks! I'll fix this in a follow-up.

         """
         fs = self.host.filesystem
         if self.default_smoke_test_only():
@@ -938,6 +938,12 @@ class Port(object):
             if fs.exists(smoke_test_filename) and test not in fs.read_text_file(smoke_test_filename):
                 return True
 
+        # In general, Skip lines in the generic expectations file indicate
+        # that the test is temporarily skipped, whereas if the test is skipped
+        # in another file (e.g. WontFix in NeverFixTests), then the test may
+        # always be skipped for this port.
+        # TODO(qyearsley): Simplify this so that it doesn't rely on having
+        # two copies of the test expectations.
         return (SKIP in full_expectations.get_expectations(test) and
                 SKIP not in generic_expectations.get_expectations(test))