PlzNavigate: identify same-page browser-initiated navigation.

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/gurl.h"
 #include "url/third_party/mozilla/url_parse.h"
@@ skipping 415 matching lines @@
     EXPECT_TRUE(IsAboutBlank(GURL(url)));
 
   const std::string kNotAboutBlankUrls[] = {
       "http:blank",      "about:blan",          "about://blank",
       "about:blank/foo", "about://:8000/blank", "about://foo:foo@/blank",
       "foo@about:blank", "foo:bar@about:blank", "about:blank:8000"};
   for (const auto& url : kNotAboutBlankUrls)
     EXPECT_FALSE(IsAboutBlank(GURL(url)));
 }
 
+TEST(URLUtilTest, EqualsIgnoringRef) {
+  const struct {
+    const char* url_a;
+    const char* url_b;
+    bool are_equals;
+  } kTestCases[] = {
+      // No ref.
+      {"http://a.com", "http://a.com", true},
+      {"http://a.com", "http://b.com", false},
+
+      // Same Ref.
+      {"http://a.com#foo", "http://a.com#foo", true},
+      {"http://a.com#foo", "http://b.com#foo", false},
+
+      // Different Refs.
+      {"http://a.com#foo", "http://a.com#bar", true},
+      {"http://a.com#foo", "http://b.com#bar", false},

[brettw, 2017/02/07 06:37:48]

Can you add a case below this of one with URL with a ref and text, and one with
no # at all?

[arthursonzogni, 2017/02/07 12:04:50]

What do you mean by "text"?
I will do as if you said: "one with a ref and the other without". In that case,
it's a good idea. Thanks!

Before committing, I'll wait for the confirmation that it is what you wanted.

+
+      // Empty refs.
+      {"http://a.com#", "http://a.com#", true},
+      {"http://a.com#", "http://a.com", true},
+      {"http://a.com", "http://a.com#", true},
+
+      // URLs that differ only by their last character.
+      {"http://aaa", "http://aab", false},
+      {"http://aaa#foo", "http://aab#foo", false},
+
+      // Different size of the part before the ref.
+      {"http://123#a", "http://123456#a", false},
+      {"http://123456#a", "http://123#a", false},
+
+      // Blob urls
+      {"blob:http://a.com#foo", "blob:http://a.com#foo", true},
+      {"blob:http://a.com#foo", "blob:http://a.com#bar", true},
+      {"blob:http://a.com#foo", "blob:http://b.com#bar", false},
+
+      // Filesystem url
+      {"filesystem:http://a.com#foo", "filesystem:http://a.com#foo", true},
+      {"filesystem:http://a.com#foo", "filesystem:http://a.com#bar", true},
+      {"filesystem:http://a.com#foo", "filesystem:http://b.com#bar", false},
+  };
+
+  for (const auto& test_case : kTestCases) {
+    SCOPED_TRACE(testing::Message()
+                 << std::endl
+                 << "url_a = " << test_case.url_a << std::endl
+                 << "url_b = " << test_case.url_b << std::endl);
+
+    EXPECT_EQ(
+        test_case.are_equals,
+        GURL::EqualsIgnoringRef(GURL(test_case.url_a), GURL(test_case.url_b)));
+  }
+}
+
 }  // namespace url