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

Unified Diff: webkit/tools/layout_tests/layout_package/test_expectations.py

Issue 146112: Allow layout tests to be located both in src/webkit/data/layout_tests and... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 6 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: webkit/tools/layout_tests/layout_package/test_expectations.py
===================================================================
--- webkit/tools/layout_tests/layout_package/test_expectations.py (revision 19531)
+++ webkit/tools/layout_tests/layout_package/test_expectations.py (working copy)
@@ -455,7 +455,8 @@
'test times out indefinitely, the it should be listed as timeout.',
test_and_expectations)
- full_path = os.path.join(path_utils.LayoutDataDir(), test_list_path)
+ full_path = os.path.join(path_utils.LayoutTestsDir(test_list_path),
+ test_list_path)
full_path = os.path.normpath(full_path)
# WebKit's way of skipping tests is to add a -disabled suffix.
# So we should consider the path existing if the path or the -disabled
@@ -504,7 +505,8 @@
def _ExpandTests(self, test_list_path):
# Convert the test specification to an absolute, normalized
# path and make sure directories end with the OS path separator.
- path = os.path.join(path_utils.LayoutDataDir(), test_list_path)
+ path = os.path.join(path_utils.LayoutTestsDir(test_list_path),
+ test_list_path)
path = os.path.normpath(path)
if os.path.isdir(path): path = os.path.join(path, '')
# This is kind of slow - O(n*m) - since this is called for all
« no previous file with comments | « webkit/tools/layout_tests/layout_package/platform_utils_win.py ('k') | webkit/tools/layout_tests/rebaseline.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698