Remove redundant IsPotentiallyTrustworthy.LegacyOriginUtilTests tests.

All of the testcases from the
IsPotentiallyTrustworthy.LegacyOriginUtilTests test were already covered
by the IsPotentiallyTrustworthy.Url test (which also covers some
additional testcases).

Bug: 1153336
Change-Id: I05f64324a40f81a7e8b52781b91946d6f859ee7d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2630297
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Frédéric Wang <fwang@igalia.com>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845818}

services/network/public/cpp/is_potentially_trustworthy_unittest.cc

@@ -186,67 +186,6 @@ TEST(IsPotentiallyTrustworthy, CustomSchemes) {
   EXPECT_FALSE(IsOriginPotentiallyTrustworthy("nonsec-noaccess-scheme:blah"));
 }
 
-// Tests that were for the removed blink::network_utils::IsOriginSecure.
-// TODO(https://crbug.com/1153336): Merge with IsPotentiallyTrustworthy.Url?
-TEST(IsPotentiallyTrustworthy, LegacyOriginUtilTests) {
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("file:///test/fun.html"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("file:///test/"));
-
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("https://example.com/fun.html"));
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy("http://example.com/fun.html"));
-
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("wss://example.com/fun.html"));
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy("ws://example.com/fun.html"));
-
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("http://localhost/fun.html"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("http://pumpkin.localhost/fun.html"));
-  EXPECT_TRUE(
-      IsUrlPotentiallyTrustworthy("http://crumpet.pumpkin.localhost/fun.html"));
-  EXPECT_TRUE(
-      IsUrlPotentiallyTrustworthy("http://pumpkin.localhost:8080/fun.html"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy(
-      "http://crumpet.pumpkin.localhost:3000/fun.html"));
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy("http://localhost.com/fun.html"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("https://localhost.com/fun.html"));
-
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("http://127.0.0.1/fun.html"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("ftp://127.0.0.1/fun.html"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("http://127.3.0.1/fun.html"));
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy("http://127.example.com/fun.html"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("https://127.example.com/fun.html"));
-
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("http://[::1]/fun.html"));
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy("http://[::2]/fun.html"));
-  EXPECT_FALSE(
-      IsUrlPotentiallyTrustworthy("http://[::1].example.com/fun.html"));
-
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy(
-      "filesystem:http://www.example.com/temporary/"));
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy(
-      "filesystem:ftp://www.example.com/temporary/"));
-  EXPECT_TRUE(
-      IsUrlPotentiallyTrustworthy("filesystem:ftp://127.0.0.1/temporary/"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy(
-      "filesystem:https://www.example.com/temporary/"));
-
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("about:blank"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("about:blank#ref"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("about:srcdoc"));
-
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy("javascript:alert('blah')"));
-
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy("data:test/plain;blah"));
-
-  EXPECT_FALSE(IsUrlPotentiallyTrustworthy(
-      "blob:http://www.example.com/guid-goes-here"));
-  EXPECT_FALSE(
-      IsUrlPotentiallyTrustworthy("blob:ftp://www.example.com/guid-goes-here"));
-  EXPECT_TRUE(
-      IsUrlPotentiallyTrustworthy("blob:ftp://127.0.0.1/guid-goes-here"));
-  EXPECT_TRUE(IsUrlPotentiallyTrustworthy(
-      "blob:https://www.example.com/guid-goes-here"));
-}
-
 class SecureOriginAllowlistTest : public testing::Test {
   void TearDown() override {
     // Ensure that we reset the allowlisted origins without any flags applied.