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

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

Issue 353983003: Making it possible to import mediastream tests. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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: 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 2bccbbd028bce16f447d9c3bdc71e7e1c8127e27..b6362ead8214cc2f555e8f01730eb3aad3b58894 100644
--- a/Tools/Scripts/webkitpy/w3c/test_importer.py
+++ b/Tools/Scripts/webkitpy/w3c/test_importer.py
@@ -41,9 +41,9 @@
overridden by a -n or --no-overwrite flag
- All files are converted to work in WebKit:
- 1. Paths to testharness.js files are modified point to Webkit's copy
- of them in LayoutTests/resources, using the correct relative path
- from the new location.
+ 1. Paths to testharness.js and vendor-prefix.js files are modified to
+ point to Webkit's copy of them in LayoutTests/resources, using the
+ correct relative path from the new location.
2. All CSS properties requiring the -webkit-vendor prefix are prefixed
(the list of what needs prefixes is read from Source/WebCore/CSS/CSSProperties.in).
3. Each reftest has its own copy of its reference file following
@@ -52,6 +52,10 @@
uses it, it is checked for paths to support files as it will be
imported into a different relative position to the test file
(in the same directory).
+ 5. Any tags with the class "instructions" have style="display:none" added
+ to them. Some w3c tests contain instructions to manual testers which we
+ want to strip out (the test result parser only recognizes pure testharness.js
+ output and not those instructions).
- Upon completion, script outputs the total number tests imported, broken
down by test type

Powered by Google App Engine
This is Rietveld 408576698