Attachment 346760 Details for Bug 187973 – WIP patch

[patch] WIP patch

wip.patch (text/plain), 3.92 KB, created by Fujii Hironori on 2018-08-08 01:26:42 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Fujii Hironori

Created: 2018-08-08 01:26:42 PDT

Size: 3.92 KB

patch

obsolete

>diff --git a/Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py b/Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py
>index aa5b511d1e0..cdcda1ac387 100644
>--- a/Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py
>+++ b/Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py
>@@ -31,7 +31,6 @@ for layout tests.
> """
> 
> import logging
>-import os.path
> import re
> 
> from webkitpy.layout_tests.models.test_configuration import TestConfigurationConverter
>@@ -139,12 +138,11 @@ class TestExpectationParser(object):
>         if not self._check_test_exists(expectation_line):
>             return
> 
>-        path = os.path.normpath(expectation_line.name)
>-        expectation_line.is_file = self._port.test_isfile(path)
>+        expectation_line.is_file = self._port.test_isfile(expectation_line.name)
>         if expectation_line.is_file:
>-            expectation_line.path = path
>+            expectation_line.path = expectation_line.name
>         else:
>-            expectation_line.path = self._port.normalize_test_name(path)
>+            expectation_line.path = self._port.normalize_test_name(expectation_line.name)
> 
>         self._collect_matching_tests(expectation_line)
> 
>diff --git a/Tools/Scripts/webkitpy/port/base.py b/Tools/Scripts/webkitpy/port/base.py
>index d607cbeb104..1c7a59c7b65 100644
>--- a/Tools/Scripts/webkitpy/port/base.py
>+++ b/Tools/Scripts/webkitpy/port/base.py
>@@ -700,10 +700,10 @@ class Port(object):
> 
>     def normalize_test_name(self, test_name):
>         """Returns a normalized version of the test name or test directory."""
>-        if test_name.endswith(os.path.sep):
>+        if test_name.endswith(self.TEST_PATH_SEPARATOR):
>             return test_name
>         if self.test_isdir(test_name):
>-            return test_name + os.path.sep
>+            return test_name + self.TEST_PATH_SEPARATOR
>         return test_name
> 
>     def driver_cmd_line_for_logging(self):
>@@ -830,7 +830,9 @@ class Port(object):
>         # Ports that run on windows need to override this method to deal with
>         # filenames with backslashes in them.
>         if filename.startswith(self.layout_tests_dir()):
>-            return self.host.filesystem.relpath(filename, self.layout_tests_dir())
>+            path = self.host.filesystem.relpath(filename, self.layout_tests_dir())
>+            path = path.replace(os.path.sep, self.TEST_PATH_SEPARATOR)
>+            return path
>         else:
>             return self.host.filesystem.abspath(filename)
> 
>diff --git a/Tools/Scripts/webkitpy/port/driver.py b/Tools/Scripts/webkitpy/port/driver.py
>index 29982adc134..ea287fe000b 100644
>--- a/Tools/Scripts/webkitpy/port/driver.py
>+++ b/Tools/Scripts/webkitpy/port/driver.py
>@@ -254,10 +254,10 @@ class Driver(object):
>             return shlex.split(self._port.get_option('wrapper')) + wrapper_arguments
>         return wrapper_arguments
> 
>-    HTTP_DIR = normpath("http/tests") + os.sep
>-    HTTP_LOCAL_DIR = normpath("http/tests/local") + os.sep
>-    WEBKIT_SPECIFIC_WEB_PLATFORM_TEST_SUBDIR = normpath("http/wpt") + os.sep
>-    WEBKIT_WEB_PLATFORM_TEST_SERVER_ROUTE = normpath("WebKit") + os.sep
>+    HTTP_DIR = "http/tests/"
>+    HTTP_LOCAL_DIR = "http/tests/local/"
>+    WEBKIT_SPECIFIC_WEB_PLATFORM_TEST_SUBDIR = "http/wpt/"
>+    WEBKIT_WEB_PLATFORM_TEST_SERVER_ROUTE = "WebKit/"
> 
>     def is_http_test(self, test_name):
>         return test_name.startswith(self.HTTP_DIR) and not test_name.startswith(self.HTTP_LOCAL_DIR)
>diff --git a/Tools/Scripts/webkitpy/port/win.py b/Tools/Scripts/webkitpy/port/win.py
>index 510b6e729c5..c86ff42aaca 100644
>--- a/Tools/Scripts/webkitpy/port/win.py
>+++ b/Tools/Scripts/webkitpy/port/win.py
>@@ -475,8 +475,6 @@ class WinCairoPort(WinPort):
> 
>     DEFAULT_ARCHITECTURE = 'x86_64'
> 
>-    TEST_PATH_SEPARATOR = os.sep
>-
>     def default_baseline_search_path(self):
>         version_name_map = VersionNameMap.map(self.host.platform)
>         if self._os_version < self.VERSION_MIN or self._os_version > self.VERSION_MAX:

Actions: View | Formatted Diff | Diff

Attachments on bug 187973: 346701 | 346702 | 346760 | 346817