Chromium Code Reviews| Index: tests/corelib_strong/uri_test.dart |
| diff --git a/tests/corelib_strong/uri_test.dart b/tests/corelib_strong/uri_test.dart |
| index f1dce7a9195618d791bc5907cbf7e44f15cf3a3d..f4b7d5ab0233bc498c22b03982becc988afd8fda 100644 |
| --- a/tests/corelib_strong/uri_test.dart |
| +++ b/tests/corelib_strong/uri_test.dart |
| @@ -213,15 +213,15 @@ void testInvalidUrls() { |
| // Success. |
| } |
| } |
| - checkInvalid("s%41://x.x/"); // No escapes in scheme, |
| + checkInvalid("s%41://x.x/"); // No escapes in scheme, |
| // and no colon before slash in path. |
| - checkInvalid("1a://x.x/"); // Scheme must start with letter, |
| + checkInvalid("1a://x.x/"); // Scheme must start with letter, |
| // and no colon before slash in path. |
| - checkInvalid(".a://x.x/"); // Scheme must start with letter, |
| + checkInvalid(".a://x.x/"); // Scheme must start with letter, |
| // and no colon before slash in path. |
| - checkInvalid("_:"); // Character not valid in scheme, |
| + checkInvalid("_:"); // Character not valid in scheme, |
| // and no colon before slash in path. |
| - checkInvalid(":"); // Scheme must start with letter, |
| + checkInvalid(":"); // Scheme must start with letter, |
| // and no colon before slash in path. |
| void checkInvalidReplaced(uri, invalid, replacement) { |
| @@ -240,20 +240,20 @@ void testInvalidUrls() { |
| // the input would cause them to be valid (normalization happens after |
| // validation). |
| var invalidCharsAndReplacements = [ |
| - "\xe7", "%C3%A7", // Arbitrary non-ASCII letter |
| - " ", "%20", // Space, not allowed anywhere. |
| - '"', "%22", // Quote, not allowed anywhere |
| - "<>", "%3C%3E", // Less/greater-than, not allowed anywhere. |
| - "\x7f", "%7F", // DEL, not allowed anywhere |
| - "\xdf", "%C3%9F", // German lower-case scharf-S. |
| + "\xe7", "%C3%A7", // Arbitrary non-ASCII letter |
| + " ", "%20", // Space, not allowed anywhere. |
| + '"', "%22", // Quote, not allowed anywhere |
| + "<>", "%3C%3E", // Less/greater-than, not allowed anywhere. |
| + "\x7f", "%7F", // DEL, not allowed anywhere |
| + "\xdf", "%C3%9F", // German lower-case scharf-S. |
| // Becomes ASCII when upper-cased. |
| - "\u0130", "%C4%B0", // Latin capital dotted I, |
| + "\u0130", "%C4%B0", // Latin capital dotted I, |
| // becomes ASCII lower-case in Turkish. |
|
sra1
2017/03/21 03:39:29
fix?
|
| "%\uFB03", "%25%EF%AC%83", // % + Ligature ffi, |
| // becomes ASCII when upper-cased, |
| // should not be read as "%FFI". |
| - "\u212a", "%E2%84%AA", // Kelvin sign. Becomes ASCII when lower-cased. |
| - "%1g", "%251g", // Invalid escape. |
| + "\u212a", "%E2%84%AA", // Kelvin sign. Becomes ASCII when lower-cased. |
| + "%1g", "%251g", // Invalid escape. |
| "\u{10000}", "%F0%90%80%80", // Non-BMP character as surrogate pair. |
| ]; |
| for (int i = 0; i < invalidCharsAndReplacements.length; i += 2) { |