Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(113)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py

Issue 1624373005: Remove XP port from Blink LayoutTest related scripts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Sync to head. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py
index 2527c8f1eae4a2e4f297dc3760b079d1cf958387..2c8955e665cd1bc7479cd14825aefb028ef947d0 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_configuration.py
@@ -258,7 +258,7 @@ class TestConfigurationConverter(object):
return False
# 2) Collapse specifier sets with common specifiers:
- # (xp, release), (xp, debug) --> (xp, x86)
+ # (win7, release), (win7, debug) --> (win7, x86)
for size, collapsing_sets in self._collapsing_sets_by_size.items():
while try_collapsing(size, collapsing_sets):
pass
@@ -278,19 +278,19 @@ class TestConfigurationConverter(object):
return False
# 3) Abbreviate specifier sets by combining specifiers across categories.
- # (xp, release), (win7, release) --> (xp, win7, release)
+ # (win7, release), (win10, release) --> (win7, win10, release)
while try_abbreviating(self._collapsing_sets_by_size.values()):
pass
# 4) Substitute specifier subsets that match macros witin each set:
- # (xp, win7, release) -> (win, release)
+ # (win7, win10, release) -> (win, release)
self.collapse_macros(self._configuration_macros, specifiers_list)
macro_keys = set(self._configuration_macros.keys())
# 5) Collapsing macros may have created combinations the can now be abbreviated.
- # (xp, release), (linux, x86, release), (linux, x86_64, release) --> (xp, release), (linux, release) --> (xp, linux, release)
+ # (win7, release), (linux, x86, release), (linux, x86_64, release) --> (win7, release), (linux, release) --> (win7, linux, release)
while try_abbreviating([self._collapsing_sets_by_category['version'] | macro_keys]):
pass

Powered by Google App Engine
This is Rietveld 408576698