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

Unified Diff: tests/corelib_strong/uri_test.dart

Issue 2763823002: Move spaces from before comments to within comments (Closed)
Patch Set: Fix comments Created 3 years, 9 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: 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) {

Powered by Google App Engine
This is Rietveld 408576698