Update dangling markup mitigations.

url/url_util_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 
 #include "base/macros.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/third_party/mozilla/url_parse.h"
 #include "url/url_canon.h"
@@ skipping 356 matching lines @@
   Parsed resolved_parsed;
 
   bool valid = ResolveRelative(base, strlen(base),
                                base_parsed, rel,
                                strlen(rel), NULL, &output,
                                &resolved_parsed);
   EXPECT_TRUE(valid);
   EXPECT_FALSE(resolved_parsed.ref.is_valid());
 }
 
-TEST(URLUtilTest, RelativeWhitespaceRemoved) {
+TEST(URLUtilTest, PotentiallyDanglingMarkup) {
   struct ResolveRelativeCase {
     const char* base;
     const char* rel;
-    bool whitespace_removed;
+    bool potentially_dangling_markup;
     const char* out;
   } cases[] = {
-      {"https://example.com/", "/path", false, "https://example.com/path"},
-      {"https://example.com/", "\n/path", true, "https://example.com/path"},
-      {"https://example.com/", "\r/path", true, "https://example.com/path"},
-      {"https://example.com/", "\t/path", true, "https://example.com/path"},
-      {"https://example.com/", "/pa\nth", true, "https://example.com/path"},
-      {"https://example.com/", "/pa\rth", true, "https://example.com/path"},
-      {"https://example.com/", "/pa\tth", true, "https://example.com/path"},
-      {"https://example.com/", "/path\n", true, "https://example.com/path"},
-      {"https://example.com/", "/path\r", true, "https://example.com/path"},
-      {"https://example.com/", "/path\r", true, "https://example.com/path"},
+      {"https://example.com/", "/path<", false, "https://example.com/path%3C"},
+      {"https://example.com/", "\n/path<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "\r/path<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "\t/path<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "/pa\nth<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "/pa\rth<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "/pa\tth<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "/path\n<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "/path\r<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "/path\r<", true, "https://example.com/path%3C"},
+      {"https://example.com/", "\n/<path", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "\r/<path", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "\t/<path", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "/<pa\nth", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "/<pa\rth", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "/<pa\tth", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "/<path\n", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "/<path\r", true, "https://example.com/%3Cpath"},
+      {"https://example.com/", "/<path\r", true, "https://example.com/%3Cpath"},
   };
 
   for (const auto& test : cases) {
     SCOPED_TRACE(::testing::Message() << test.base << ", " << test.rel);
     Parsed base_parsed;
     ParseStandardURL(test.base, strlen(test.base), &base_parsed);
 
     std::string resolved;
     StdStringCanonOutput output(&resolved);
     Parsed resolved_parsed;
     bool valid =
         ResolveRelative(test.base, strlen(test.base), base_parsed, test.rel,
                         strlen(test.rel), NULL, &output, &resolved_parsed);
     ASSERT_TRUE(valid);
     output.Complete();
 
-    EXPECT_EQ(test.whitespace_removed, resolved_parsed.whitespace_removed);
+    EXPECT_EQ(test.potentially_dangling_markup,
+              resolved_parsed.potentially_dangling_markup);
     EXPECT_EQ(test.out, resolved);
   }
 }
 
 TEST(URLUtilTest, TestDomainIs) {
   const struct {
     const char* canonicalized_host;
     const char* lower_ascii_domain;
     bool expected_domain_is;
   } kTestCases[] = {
@@ skipping 28 matching lines @@
                                     << test_case.canonicalized_host << ", "
                                     << test_case.lower_ascii_domain << ")");
 
     EXPECT_EQ(
         test_case.expected_domain_is,
         DomainIs(test_case.canonicalized_host, test_case.lower_ascii_domain));
   }
 }
 
 }  // namespace url