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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/encoding/textdecoder-labels.html

Issue 2657583002: Import wpt@cf62b859e6b890abc34f8140d185ba91df95c5b6 (Closed)
Patch Set: Modify TestExpectations or download new baselines for tests. Created 3 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/LayoutTests/external/wpt/encoding/textdecoder-labels.html
diff --git a/third_party/WebKit/LayoutTests/external/wpt/encoding/textdecoder-labels.html b/third_party/WebKit/LayoutTests/external/wpt/encoding/textdecoder-labels.html
index 1f3202c963eb2242a0edcf8c3d363c134524030f..04ae0266d91c38086464a631f10f4224935a055a 100644
--- a/third_party/WebKit/LayoutTests/external/wpt/encoding/textdecoder-labels.html
+++ b/third_party/WebKit/LayoutTests/external/wpt/encoding/textdecoder-labels.html
@@ -4,33 +4,32 @@
<script src="/resources/testharnessreport.js"></script>
<script src="resources/encodings.js"></script>
<script>
-var tests = [];
-setup(function() {
- var whitespace = [' ', '\t', '\n', '\f', '\r'];
- encodings_table.forEach(function(section) {
- section.encodings.filter(function(encoding) {
- return encoding.name !== 'replacement';
- }).forEach(function(encoding) {
- var name = encoding.name;
- encoding.labels.forEach(function(label) {
- tests.push([label, encoding.name]);
+var whitespace = [' ', '\t', '\n', '\f', '\r'];
+encodings_table.forEach(function(section) {
+ section.encodings.filter(function(encoding) {
+ return encoding.name !== 'replacement';
+ }).forEach(function(encoding) {
+ encoding.labels.forEach(function(label) {
+ test(function(t) {
+ assert_equals(
+ new TextDecoder(label).encoding, encoding.name,
+ 'label for encoding should match');
+ assert_equals(
+ new TextDecoder(label.toUpperCase()).encoding, encoding.name,
+ 'label matching should be case-insensitive');
whitespace.forEach(function(ws) {
- tests.push([ws + label, encoding.name]);
- tests.push([label + ws, encoding.name]);
- tests.push([ws + label + ws, encoding.name]);
+ assert_equals(
+ new TextDecoder(ws + label).encoding, encoding.name,
+ 'label for encoding with leading whitespace should match');
+ assert_equals(
+ new TextDecoder(label + ws).encoding, encoding.name,
+ 'label for encoding with trailing whitespace should match');
+ assert_equals(
+ new TextDecoder(ws + label + ws).encoding, encoding.name,
+ 'label for encoding with surrounding whitespace should match');
});
- });
+ }, label + ' => ' + encoding.name);
});
});
});
-
-tests.forEach(function(t) {
- var input = t[0], output = t[1];
- test(function() {
- assert_equals(new TextDecoder(input).encoding, output,
- 'label for encoding should match');
- assert_equals(new TextDecoder(input.toUpperCase()).encoding, output,
- 'label matching should be case-insensitive');
- }, format_value(input) + " => " + format_value(output));
-});
</script>

Powered by Google App Engine
This is Rietveld 408576698