Changeset 243769 in webkit
- Timestamp:
- Apr 2, 2019, 5:03:56 PM (7 years ago)
- Location:
- branches/safari-607-branch/Tools
- Files:
-
- 10 edited
-
ChangeLog (modified) (1 diff)
-
Scripts/webkitpy/port/base.py (modified) (1 diff)
-
Scripts/webkitpy/port/gtk.py (modified) (1 diff)
-
Scripts/webkitpy/port/ios.py (modified) (3 diffs)
-
Scripts/webkitpy/port/ios_device_unittest.py (modified) (2 diffs)
-
Scripts/webkitpy/port/ios_simulator_unittest.py (modified) (1 diff)
-
Scripts/webkitpy/port/mac.py (modified) (1 diff)
-
Scripts/webkitpy/port/watch.py (modified) (1 diff)
-
Scripts/webkitpy/port/win.py (modified) (2 diffs)
-
Scripts/webkitpy/port/wpe.py (modified) (1 diff)
Legend:
- Unmodified
- Added
- Removed
-
branches/safari-607-branch/Tools/ChangeLog
r243748 r243769 1 2019-04-02 Kocsen Chung <kocsen_chung@apple.com> 2 3 Cherry-pick r239875. rdar://problem/49539128 4 5 webkitpy: Incorporate device type into baseline search path 6 https://bugs.webkit.org/show_bug.cgi?id=193356 7 <rdar://problem/47215515> 8 9 Reviewed by Lucas Forschler. 10 11 We need a way to include device type in the baseline search path for iOS to support device specific test expectations. 12 13 * Scripts/webkitpy/port/base.py: 14 (Port.default_baseline_search_path): Allow device type to be passed in. 15 * Scripts/webkitpy/port/gtk.py: 16 (GtkPort.default_baseline_search_path): Ignore device type. 17 * Scripts/webkitpy/port/ios.py: 18 (IOSPort.default_baseline_search_path): Optionally allow device type to be incorporated into the baseline search path. 19 (IOSPort.test_expectations_file_position): Update index for the additional device-type specific baseline search path. 20 * Scripts/webkitpy/port/ios_device_unittest.py: 21 (IOSDeviceTest): 22 * Scripts/webkitpy/port/ios_simulator_unittest.py: 23 (IOSSimulatorTest): 24 * Scripts/webkitpy/port/mac.py: 25 (MacPort.default_baseline_search_path): Ignore device type. 26 * Scripts/webkitpy/port/watch.py: 27 (WatchPort.default_baseline_search_path): Ditto. 28 * Scripts/webkitpy/port/win.py: 29 (WinPort.default_baseline_search_path): Ditto. 30 (WinCairoPort.default_baseline_search_path): Ditto. 31 * Scripts/webkitpy/port/wpe.py: 32 (WPEPort.default_baseline_search_path): Ditto. 33 34 35 git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239875 268f45cc-cd09-0410-ab3c-d52691b4dbfc 36 37 2019-01-11 Jonathan Bedard <jbedard@apple.com> 38 39 webkitpy: Incorporate device type into baseline search path 40 https://bugs.webkit.org/show_bug.cgi?id=193356 41 <rdar://problem/47215515> 42 43 Reviewed by Lucas Forschler. 44 45 We need a way to include device type in the baseline search path for iOS to support device specific test expectations. 46 47 * Scripts/webkitpy/port/base.py: 48 (Port.default_baseline_search_path): Allow device type to be passed in. 49 * Scripts/webkitpy/port/gtk.py: 50 (GtkPort.default_baseline_search_path): Ignore device type. 51 * Scripts/webkitpy/port/ios.py: 52 (IOSPort.default_baseline_search_path): Optionally allow device type to be incorporated into the baseline search path. 53 (IOSPort.test_expectations_file_position): Update index for the additional device-type specific baseline search path. 54 * Scripts/webkitpy/port/ios_device_unittest.py: 55 (IOSDeviceTest): 56 * Scripts/webkitpy/port/ios_simulator_unittest.py: 57 (IOSSimulatorTest): 58 * Scripts/webkitpy/port/mac.py: 59 (MacPort.default_baseline_search_path): Ignore device type. 60 * Scripts/webkitpy/port/watch.py: 61 (WatchPort.default_baseline_search_path): Ditto. 62 * Scripts/webkitpy/port/win.py: 63 (WinPort.default_baseline_search_path): Ditto. 64 (WinCairoPort.default_baseline_search_path): Ditto. 65 * Scripts/webkitpy/port/wpe.py: 66 (WPEPort.default_baseline_search_path): Ditto. 67 1 68 2019-04-02 Kocsen Chung <kocsen_chung@apple.com> 2 69 -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/base.py
r243748 r243769 214 214 return self.get_option('additional_platform_directory', []) + self._compare_baseline() + self.default_baseline_search_path() 215 215 216 def default_baseline_search_path(self ):216 def default_baseline_search_path(self, device_type=None): 217 217 """Return a list of absolute paths to directories to search under for 218 218 baselines. The directories are searched in order.""" -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/gtk.py
r238749 r243769 211 211 return search_paths 212 212 213 def default_baseline_search_path(self ):213 def default_baseline_search_path(self, **kwargs): 214 214 return map(self._webkit_baseline_path, self._search_paths()) 215 215 -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/ios.py
r238749 r243769 54 54 55 55 @memoized 56 def default_baseline_search_path(self ):56 def default_baseline_search_path(self, device_type=None): 57 57 wk_string = 'wk1' 58 58 if self.get_option('webkit_test_runner'): … … 71 71 temp_version.major -= 1 72 72 73 runtime_type = 'simulator' if 'simulator' in self.SDK else 'device' 74 hardware_family = device_type.hardware_family.lower() if device_type and device_type.hardware_family else None 75 hardware_type = device_type.hardware_type.lower() if device_type and device_type.hardware_type else None 76 77 base_variants = [] 78 if hardware_family and hardware_type: 79 base_variants.append('{}-{}-{}'.format(hardware_family, hardware_type, runtime_type)) 80 if hardware_family: 81 base_variants.append('{}-{}'.format(hardware_family, runtime_type)) 82 base_variants.append('{}-{}'.format(IOSPort.port_name, runtime_type)) 83 if hardware_family and hardware_type: 84 base_variants.append('{}-{}'.format(hardware_family, hardware_type)) 85 if hardware_family: 86 base_variants.append(hardware_family) 87 base_variants.append(IOSPort.port_name) 88 73 89 expectations = [] 74 for version in versions_to_fallback: 75 apple_name = None 90 for variant in base_variants: 91 for version in versions_to_fallback: 92 apple_name = None 93 if apple_additions(): 94 apple_name = VersionNameMap.map(self.host.platform).to_name(version, platform=IOSPort.port_name, table=INTERNAL_TABLE) 95 96 if apple_name: 97 expectations.append(self._apple_baseline_path('{}-{}-{}'.format(variant, apple_name.lower().replace(' ', ''), wk_string))) 98 expectations.append(self._webkit_baseline_path('{}-{}-{}'.format(variant, version.major, wk_string))) 99 if apple_name: 100 expectations.append(self._apple_baseline_path('{}-{}'.format(variant, apple_name.lower().replace(' ', '')))) 101 expectations.append(self._webkit_baseline_path('{}-{}'.format(variant, version.major))) 102 76 103 if apple_additions(): 77 apple_name = VersionNameMap.map(self.host.platform).to_name(version, platform=IOSPort.port_name, table=INTERNAL_TABLE) 78 79 if apple_name: 80 expectations.append(self._apple_baseline_path('{}-{}-{}'.format(self.port_name, apple_name.lower().replace(' ', ''), wk_string))) 81 expectations.append(self._webkit_baseline_path('{}-{}-{}'.format(self.port_name, version.major, wk_string))) 82 if apple_name: 83 expectations.append(self._apple_baseline_path('{}-{}'.format(self.port_name, apple_name.lower().replace(' ', '')))) 84 expectations.append(self._webkit_baseline_path('{}-{}'.format(self.port_name, version.major))) 85 86 if apple_additions(): 87 expectations.append(self._apple_baseline_path('{}-{}'.format(self.port_name, wk_string))) 88 expectations.append(self._webkit_baseline_path('{}-{}'.format(self.port_name, wk_string))) 89 if apple_additions(): 90 expectations.append(self._apple_baseline_path(self.port_name)) 91 expectations.append(self._webkit_baseline_path(self.port_name)) 92 93 for version in versions_to_fallback: 94 apple_name = None 104 expectations.append(self._apple_baseline_path('{}-{}'.format(variant, wk_string))) 105 expectations.append(self._webkit_baseline_path('{}-{}'.format(variant, wk_string))) 95 106 if apple_additions(): 96 apple_name = VersionNameMap.map(self.host.platform).to_name(version, platform=IOSPort.port_name, table=INTERNAL_TABLE) 97 if apple_name: 98 expectations.append(self._apple_baseline_path('{}-{}'.format(IOSPort.port_name, apple_name.lower().replace(' ', '')))) 99 expectations.append(self._webkit_baseline_path('{}-{}'.format(IOSPort.port_name, version.major))) 100 101 if apple_additions(): 102 expectations.append(self._apple_baseline_path('{}-{}'.format(IOSPort.port_name, wk_string))) 103 expectations.append(self._webkit_baseline_path('{}-{}'.format(IOSPort.port_name, wk_string))) 104 if apple_additions(): 105 expectations.append(self._apple_baseline_path(IOSPort.port_name)) 106 expectations.append(self._webkit_baseline_path(IOSPort.port_name)) 107 expectations.append(self._apple_baseline_path(variant)) 108 expectations.append(self._webkit_baseline_path(variant)) 107 109 108 110 if self.get_option('webkit_test_runner'): … … 112 114 113 115 def test_expectations_file_position(self): 114 return 4116 return 5 -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/ios_device_unittest.py
r238944 r243769 29 29 from webkitpy.port import ios_testcase 30 30 from webkitpy.port import port_testcase 31 from webkitpy.xcode.device_type import DeviceType 31 32 32 33 … … 89 90 with port_testcase.bind_mock_apple_additions(): 90 91 search_path = self.make_port().default_baseline_search_path() 91 self.assertEqual(search_path[0], '/additional_testing_path/ios-device-add-ios11-wk1') 92 self.assertEqual(search_path[1], '/mock-checkout/LayoutTests/platform/ios-device-11-wk1') 93 self.assertEqual(search_path[2], '/additional_testing_path/ios-device-add-ios11') 94 self.assertEqual(search_path[3], '/mock-checkout/LayoutTests/platform/ios-device-11') 95 self.assertEqual(search_path[4], '/additional_testing_path/ios-device-wk1') 96 self.assertEqual(search_path[5], '/mock-checkout/LayoutTests/platform/ios-device-wk1') 97 self.assertEqual(search_path[6], '/additional_testing_path/ios-device') 98 self.assertEqual(search_path[7], '/mock-checkout/LayoutTests/platform/ios-device') 99 self.assertEqual(search_path[8], '/additional_testing_path/ios-add-ios11') 100 self.assertEqual(search_path[9], '/mock-checkout/LayoutTests/platform/ios-11') 101 self.assertEqual(search_path[10], '/additional_testing_path/ios-wk1') 102 self.assertEqual(search_path[11], '/mock-checkout/LayoutTests/platform/ios-wk1') 92 93 self.assertEqual(search_path, [ 94 '/additional_testing_path/ios-device-add-ios11-wk1', 95 '/mock-checkout/LayoutTests/platform/ios-device-11-wk1', 96 '/additional_testing_path/ios-device-add-ios11', 97 '/mock-checkout/LayoutTests/platform/ios-device-11', 98 '/additional_testing_path/ios-device-wk1', 99 '/mock-checkout/LayoutTests/platform/ios-device-wk1', 100 '/additional_testing_path/ios-device', 101 '/mock-checkout/LayoutTests/platform/ios-device', 102 '/additional_testing_path/ios-add-ios11-wk1', 103 '/mock-checkout/LayoutTests/platform/ios-11-wk1', 104 '/additional_testing_path/ios-add-ios11', 105 '/mock-checkout/LayoutTests/platform/ios-11', 106 '/additional_testing_path/ios-wk1', 107 '/mock-checkout/LayoutTests/platform/ios-wk1', 108 '/additional_testing_path/ios', 109 '/mock-checkout/LayoutTests/platform/ios', 110 ]) 103 111 104 112 def test_layout_test_searchpath_without_apple_additions(self): 105 113 search_path = self.make_port(port_name='ios-device-wk2', os_version=Version(12)).default_baseline_search_path() 106 114 107 self.assertEqual(search_path[0], '/mock-checkout/LayoutTests/platform/ios-device-12-wk2') 108 self.assertEqual(search_path[1], '/mock-checkout/LayoutTests/platform/ios-device-12') 109 self.assertEqual(search_path[2], '/mock-checkout/LayoutTests/platform/ios-device-wk2') 110 self.assertEqual(search_path[3], '/mock-checkout/LayoutTests/platform/ios-device') 111 self.assertEqual(search_path[4], '/mock-checkout/LayoutTests/platform/ios-12') 112 self.assertEqual(search_path[5], '/mock-checkout/LayoutTests/platform/ios-wk2') 113 self.assertEqual(search_path[6], '/mock-checkout/LayoutTests/platform/ios') 114 self.assertEqual(search_path[7], '/mock-checkout/LayoutTests/platform/wk2') 115 self.assertEqual(search_path, [ 116 '/mock-checkout/LayoutTests/platform/ios-device-12-wk2', 117 '/mock-checkout/LayoutTests/platform/ios-device-12', 118 '/mock-checkout/LayoutTests/platform/ios-device-wk2', 119 '/mock-checkout/LayoutTests/platform/ios-device', 120 '/mock-checkout/LayoutTests/platform/ios-12-wk2', 121 '/mock-checkout/LayoutTests/platform/ios-12', 122 '/mock-checkout/LayoutTests/platform/ios-wk2', 123 '/mock-checkout/LayoutTests/platform/ios', 124 '/mock-checkout/LayoutTests/platform/wk2', 125 ]) 126 127 def test_layout_searchpath_wih_device_type(self): 128 search_path = self.make_port(port_name='ios-device-wk2', os_version=Version(12)).default_baseline_search_path(DeviceType.from_string('iPhone SE')) 129 130 self.assertEqual(search_path, [ 131 '/mock-checkout/LayoutTests/platform/iphone-se-device-12-wk2', 132 '/mock-checkout/LayoutTests/platform/iphone-se-device-12', 133 '/mock-checkout/LayoutTests/platform/iphone-se-device-wk2', 134 '/mock-checkout/LayoutTests/platform/iphone-se-device', 135 '/mock-checkout/LayoutTests/platform/iphone-device-12-wk2', 136 '/mock-checkout/LayoutTests/platform/iphone-device-12', 137 '/mock-checkout/LayoutTests/platform/iphone-device-wk2', 138 '/mock-checkout/LayoutTests/platform/iphone-device', 139 '/mock-checkout/LayoutTests/platform/ios-device-12-wk2', 140 '/mock-checkout/LayoutTests/platform/ios-device-12', 141 '/mock-checkout/LayoutTests/platform/ios-device-wk2', 142 '/mock-checkout/LayoutTests/platform/ios-device', 143 '/mock-checkout/LayoutTests/platform/iphone-se-12-wk2', 144 '/mock-checkout/LayoutTests/platform/iphone-se-12', 145 '/mock-checkout/LayoutTests/platform/iphone-se-wk2', 146 '/mock-checkout/LayoutTests/platform/iphone-se', 147 '/mock-checkout/LayoutTests/platform/iphone-12-wk2', 148 '/mock-checkout/LayoutTests/platform/iphone-12', 149 '/mock-checkout/LayoutTests/platform/iphone-wk2', 150 '/mock-checkout/LayoutTests/platform/iphone', 151 '/mock-checkout/LayoutTests/platform/ios-12-wk2', 152 '/mock-checkout/LayoutTests/platform/ios-12', 153 '/mock-checkout/LayoutTests/platform/ios-wk2', 154 '/mock-checkout/LayoutTests/platform/ios', 155 '/mock-checkout/LayoutTests/platform/wk2', 156 ]) 115 157 116 158 def test_max_child_processes(self): -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/ios_simulator_unittest.py
r243748 r243769 93 93 with port_testcase.bind_mock_apple_additions(): 94 94 search_path = self.make_port().default_baseline_search_path() 95 self.assertEqual(search_path[0], '/additional_testing_path/ios-simulator-add-ios11-wk1') 96 self.assertEqual(search_path[1], '/mock-checkout/LayoutTests/platform/ios-simulator-11-wk1') 97 self.assertEqual(search_path[2], '/additional_testing_path/ios-simulator-add-ios11') 98 self.assertEqual(search_path[3], '/mock-checkout/LayoutTests/platform/ios-simulator-11') 99 self.assertEqual(search_path[4], '/additional_testing_path/ios-simulator-wk1') 100 self.assertEqual(search_path[5], '/mock-checkout/LayoutTests/platform/ios-simulator-wk1') 101 self.assertEqual(search_path[6], '/additional_testing_path/ios-simulator') 102 self.assertEqual(search_path[7], '/mock-checkout/LayoutTests/platform/ios-simulator') 103 self.assertEqual(search_path[8], '/additional_testing_path/ios-add-ios11') 104 self.assertEqual(search_path[9], '/mock-checkout/LayoutTests/platform/ios-11') 105 self.assertEqual(search_path[10], '/additional_testing_path/ios-wk1') 106 self.assertEqual(search_path[11], '/mock-checkout/LayoutTests/platform/ios-wk1') 95 96 self.assertEqual(search_path, [ 97 '/additional_testing_path/ios-simulator-add-ios11-wk1', 98 '/mock-checkout/LayoutTests/platform/ios-simulator-11-wk1', 99 '/additional_testing_path/ios-simulator-add-ios11', 100 '/mock-checkout/LayoutTests/platform/ios-simulator-11', 101 '/additional_testing_path/ios-simulator-wk1', 102 '/mock-checkout/LayoutTests/platform/ios-simulator-wk1', 103 '/additional_testing_path/ios-simulator', 104 '/mock-checkout/LayoutTests/platform/ios-simulator', 105 '/additional_testing_path/ios-add-ios11-wk1', 106 '/mock-checkout/LayoutTests/platform/ios-11-wk1', 107 '/additional_testing_path/ios-add-ios11', 108 '/mock-checkout/LayoutTests/platform/ios-11', 109 '/additional_testing_path/ios-wk1', 110 '/mock-checkout/LayoutTests/platform/ios-wk1', 111 '/additional_testing_path/ios', 112 '/mock-checkout/LayoutTests/platform/ios', 113 ]) 107 114 108 115 def test_layout_test_searchpath_without_apple_additions(self): 109 116 search_path = self.make_port(port_name='ios-simulator-wk2', os_version=Version(12)).default_baseline_search_path() 110 117 111 self.assertEqual(search_path[0], '/mock-checkout/LayoutTests/platform/ios-simulator-12-wk2') 112 self.assertEqual(search_path[1], '/mock-checkout/LayoutTests/platform/ios-simulator-12') 113 self.assertEqual(search_path[2], '/mock-checkout/LayoutTests/platform/ios-simulator-wk2') 114 self.assertEqual(search_path[3], '/mock-checkout/LayoutTests/platform/ios-simulator') 115 self.assertEqual(search_path[4], '/mock-checkout/LayoutTests/platform/ios-12') 116 self.assertEqual(search_path[5], '/mock-checkout/LayoutTests/platform/ios-wk2') 117 self.assertEqual(search_path[6], '/mock-checkout/LayoutTests/platform/ios') 118 self.assertEqual(search_path[7], '/mock-checkout/LayoutTests/platform/wk2') 118 self.assertEqual(search_path, [ 119 '/mock-checkout/LayoutTests/platform/ios-simulator-12-wk2', 120 '/mock-checkout/LayoutTests/platform/ios-simulator-12', 121 '/mock-checkout/LayoutTests/platform/ios-simulator-wk2', 122 '/mock-checkout/LayoutTests/platform/ios-simulator', 123 '/mock-checkout/LayoutTests/platform/ios-12-wk2', 124 '/mock-checkout/LayoutTests/platform/ios-12', 125 '/mock-checkout/LayoutTests/platform/ios-wk2', 126 '/mock-checkout/LayoutTests/platform/ios', 127 '/mock-checkout/LayoutTests/platform/wk2', 128 ]) 129 130 def test_layout_searchpath_wih_device_type(self): 131 search_path = self.make_port(port_name='ios-simulator-wk2', os_version=Version(12)).default_baseline_search_path(DeviceType.from_string('iPhone SE')) 132 133 self.assertEqual(search_path, [ 134 '/mock-checkout/LayoutTests/platform/iphone-se-simulator-12-wk2', 135 '/mock-checkout/LayoutTests/platform/iphone-se-simulator-12', 136 '/mock-checkout/LayoutTests/platform/iphone-se-simulator-wk2', 137 '/mock-checkout/LayoutTests/platform/iphone-se-simulator', 138 '/mock-checkout/LayoutTests/platform/iphone-simulator-12-wk2', 139 '/mock-checkout/LayoutTests/platform/iphone-simulator-12', 140 '/mock-checkout/LayoutTests/platform/iphone-simulator-wk2', 141 '/mock-checkout/LayoutTests/platform/iphone-simulator', 142 '/mock-checkout/LayoutTests/platform/ios-simulator-12-wk2', 143 '/mock-checkout/LayoutTests/platform/ios-simulator-12', 144 '/mock-checkout/LayoutTests/platform/ios-simulator-wk2', 145 '/mock-checkout/LayoutTests/platform/ios-simulator', 146 '/mock-checkout/LayoutTests/platform/iphone-se-12-wk2', 147 '/mock-checkout/LayoutTests/platform/iphone-se-12', 148 '/mock-checkout/LayoutTests/platform/iphone-se-wk2', 149 '/mock-checkout/LayoutTests/platform/iphone-se', 150 '/mock-checkout/LayoutTests/platform/iphone-12-wk2', 151 '/mock-checkout/LayoutTests/platform/iphone-12', 152 '/mock-checkout/LayoutTests/platform/iphone-wk2', 153 '/mock-checkout/LayoutTests/platform/iphone', 154 '/mock-checkout/LayoutTests/platform/ios-12-wk2', 155 '/mock-checkout/LayoutTests/platform/ios-12', 156 '/mock-checkout/LayoutTests/platform/ios-wk2', 157 '/mock-checkout/LayoutTests/platform/ios', 158 '/mock-checkout/LayoutTests/platform/wk2', 159 ]) 119 160 120 161 def test_max_child_processes(self): -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/mac.py
r243748 r243769 71 71 72 72 @memoized 73 def default_baseline_search_path(self ):73 def default_baseline_search_path(self, **kwargs): 74 74 versions_to_fallback = [] 75 75 version_name_map = VersionNameMap.map(self.host.platform) -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/watch.py
r238749 r243769 60 60 61 61 @memoized 62 def default_baseline_search_path(self ):62 def default_baseline_search_path(self, **kwargs): 63 63 versions_to_fallback = [] 64 64 if self.device_version() == self.CURRENT_VERSION: -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/win.py
r238749 r243769 111 111 return expected_text != actual_text 112 112 113 def default_baseline_search_path(self ):113 def default_baseline_search_path(self, **kwargs): 114 114 version_name_map = VersionNameMap.map(self.host.platform) 115 115 if self._os_version < self.VERSION_MIN or self._os_version > self.VERSION_MAX: … … 481 481 DEFAULT_ARCHITECTURE = 'x86_64' 482 482 483 def default_baseline_search_path(self ):483 def default_baseline_search_path(self, **kwargs): 484 484 version_name_map = VersionNameMap.map(self.host.platform) 485 485 if self._os_version < self.VERSION_MIN or self._os_version > self.VERSION_MAX: -
branches/safari-607-branch/Tools/Scripts/webkitpy/port/wpe.py
r237741 r243769 103 103 return [self.port_name, 'wk2'] + self.get_option("additional_platform_directory", []) 104 104 105 def default_baseline_search_path(self ):105 def default_baseline_search_path(self, **kwargs): 106 106 return map(self._webkit_baseline_path, self._search_paths()) 107 107
Note:
See TracChangeset
for help on using the changeset viewer.