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

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

Issue 15366004: update w3c import script to actually work :). (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_parser.py
diff --git a/Tools/Scripts/webkitpy/w3c/test_parser.py b/Tools/Scripts/webkitpy/w3c/test_parser.py
index bdb24dfd656da1dbe11bedd8adcbe49743633313..8f227ee7f717c7f7aa5d6c8bb673af3cc457720c 100644
--- a/Tools/Scripts/webkitpy/w3c/test_parser.py
+++ b/Tools/Scripts/webkitpy/w3c/test_parser.py
@@ -47,7 +47,7 @@ class TestParser(object):
def load_file(self, filename):
if self.filesystem.exists(filename):
- self.test_doc = Parser(self.filesystem.read_text_file(filename))
+ self.test_doc = Parser(self.filesystem.read_binary_file(filename))
else:
self.test_doc = None
self.ref_doc = None
@@ -92,7 +92,7 @@ class TestParser(object):
# support files.
#
# *But*, there is exactly one case in the entire css2.1 suite where
- # at test depends on a file that lives in a different directory,
+ # a test depends on a file that lives in a different directory,
# which depends on another file that lives outside of its
# directory. This code covers that case :)
if matches[0]['href'].startswith('..'):
@@ -127,8 +127,8 @@ class TestParser(object):
urls = []
for url in doc.findAll(text=url_pattern):
url = re.search(url_pattern, url)
- url = re.sub('url\([\'\"]', '', url.group(0))
- url = re.sub('[\'\"]\)', '', url)
+ url = re.sub('url\([\'\"]?', '', url.group(0))
+ url = re.sub('[\'\"]?\)', '', url)
urls.append(url)
src_paths = [src_tag['src'] for src_tag in elements_with_src_attributes]
« Tools/Scripts/webkitpy/w3c/test_converter.py ('K') | « Tools/Scripts/webkitpy/w3c/test_importer.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698