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

Unified Diff: Tools/Scripts/webkitpy/w3c/test_importer.py

Issue 15592005: support .xht directly so we do not have to rename files during w3c imports (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 7 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: Tools/Scripts/webkitpy/w3c/test_importer.py
diff --git a/Tools/Scripts/webkitpy/w3c/test_importer.py b/Tools/Scripts/webkitpy/w3c/test_importer.py
index 0dbd823818209a6fc618a66a2ce58804b9db50b9..71494b9ff3d42e34f04f9199d1a0138c0895311a 100644
--- a/Tools/Scripts/webkitpy/w3c/test_importer.py
+++ b/Tools/Scripts/webkitpy/w3c/test_importer.py
@@ -65,7 +65,6 @@
This can also be overridden by a -n or --no-overwrite flag
- All files are converted to work in WebKit:
- 1. .xht extensions are changed to .xhtml to make new-run-webkit-tests happy
2. Paths to testharness.js files are modified point to Webkit's copy of them in
ojan 2013/05/22 00:22:06 Numbering needs updating.
LayoutTests/resources, using the correct relative path from the new location
3. All CSS properties requiring the -webkit-vendor prefix are prefixed - this current
@@ -303,9 +302,6 @@ class TestImporter(object):
new_filepath = os.path.join(new_path, file_to_copy['dest'])
- # FIXME: we should just support '.xht' directly.
- new_filepath = new_filepath.replace('.xht', '.xhtml')
-
if not(os.path.exists(os.path.dirname(new_filepath))):
os.makedirs(os.path.dirname(new_filepath))

Powered by Google App Engine
This is Rietveld 408576698