Add removal filter support for Cookies, Storage, and Content Settings.

chrome/browser/browsing_data/browsing_data_filter_builder_unittest.cc

+// Copyright 2015 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 "chrome/browser/browsing_data/browsing_data_filter_builder.h"
+
+#include <algorithm>
+#include <string>
+#include <vector>
+
+#include "base/callback.h"
+#include "components/content_settings/core/common/content_settings_pattern.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "url/gurl.h"
+#include "url/origin.h"
+
+namespace {
+const char kGoogleDomain[] = "google.com";
+// sp.nom.br is an eTLD, so this is a regular valid registrable domain, just
+// like google.com.
+const char kLongETLDDomain[] = "website.sp.nom.br";
+// An IP address can be a registerable domain.
+const char kIPAddress[] = "192.168.1.1";
+
+struct TestCase {
+  std::string url;
+  bool should_match;
+};
+
+void RunTestCase(TestCase test_case,
+                 const base::Callback<bool(const GURL&)>& filter) {
+  GURL url(test_case.url);
+  EXPECT_TRUE(url.is_valid()) << test_case.url << " is not valid.";
+  EXPECT_EQ(test_case.should_match, filter.Run(GURL(test_case.url)))
+      << test_case.url;
+}
+
+void RunTestCase(
+    TestCase test_case,
+    const base::Callback<bool(const ContentSettingsPattern&)>& filter) {
+  ContentSettingsPattern pattern =
+      ContentSettingsPattern::FromString(test_case.url);
+  EXPECT_TRUE(pattern.IsValid()) << test_case.url << " is not valid.";
+  EXPECT_EQ(test_case.should_match, filter.Run(pattern)) << pattern.ToString();
+}
+
+void RunTestCase(
+    TestCase test_case,
+    const base::Callback<bool(const net::CanonicalCookie&)>& filter) {
+  // Test with regular cookie, http only, domain, and secure.
+  std::string cookie_line = "A=2";
+  GURL test_url(test_case.url);
+  EXPECT_TRUE(test_url.is_valid()) << test_case.url;
+  scoped_ptr<net::CanonicalCookie> cookie = net::CanonicalCookie::Create(
+      test_url, cookie_line, base::Time::Now(), net::CookieOptions());
+  EXPECT_TRUE(cookie) << cookie_line << " from " << test_case.url
+                      << " is not a valid cookie";
+  if (cookie)
+    EXPECT_EQ(test_case.should_match, filter.Run(*cookie))
+        << cookie->DebugString();
+
+  cookie_line = std::string("A=2;domain=") + test_url.host();
+  cookie = net::CanonicalCookie::Create(
+      test_url, cookie_line, base::Time::Now(), net::CookieOptions());
+  if (cookie)
+    EXPECT_EQ(test_case.should_match, filter.Run(*cookie))
+        << cookie->DebugString();
+
+  cookie_line = std::string("A=2; HttpOnly;") + test_url.host();
+  cookie = net::CanonicalCookie::Create(
+      test_url, cookie_line, base::Time::Now(), net::CookieOptions());
+  if (cookie)
+    EXPECT_EQ(test_case.should_match, filter.Run(*cookie))
+        << cookie->DebugString();
+
+  cookie_line = std::string("A=2; HttpOnly; Secure;") + test_url.host();
+  cookie = net::CanonicalCookie::Create(
+      test_url, cookie_line, base::Time::Now(), net::CookieOptions());
+  if (cookie)
+    EXPECT_EQ(test_case.should_match, filter.Run(*cookie))
+        << cookie->DebugString();
+}
+
+}  // namespace
+
+TEST(BrowsingDataFilterBuilderTest, Noop) {
+  // An no-op filter matches everything.
+  base::Callback<bool(const GURL&)> filter =
+      BrowsingDataFilterBuilder::BuildNoopFilter();
+
+  TestCase test_cases[] = {
+      {"https://www.google.com", true},
+      {"https://www.chrome.com", true},
+      {"http://www.google.com/foo/bar", true},
+      {"https://website.sp.nom.br", true},
+  };
+
+  for (TestCase test_case : test_cases)
+    RunTestCase(test_case, filter);
+}
+
+TEST(BrowsingDataFilterBuilderTest, GURLWhitelist) {
+  BrowsingDataFilterBuilder builder(BrowsingDataFilterBuilder::WHITELIST);
+  builder.AddRegisterableDomain(std::string(kGoogleDomain));
+  builder.AddRegisterableDomain(std::string(kLongETLDDomain));
+  builder.AddRegisterableDomain(std::string(kIPAddress));
+  base::Callback<bool(const GURL&)> filter = builder.BuildSameDomainFilter();
+
+  TestCase test_cases[] = {
+      // We matche any URL on the specified domains.
+      {"http://www.google.com/foo/bar", true},
+      {"https://www.sub.google.com/foo/bar", true},
+      {"https://sub.google.com", true},
+      {"http://www.sub.google.com:8000/foo/bar", true},
+      {"https://website.sp.nom.br", true},
+      {"https://www.website.sp.nom.br", true},
+      {"http://192.168.1.1", true},
+      {"http://192.168.1.1:80", true},
+
+      // Different domains.
+      {"https://www.youtube.com", false},
+      {"https://www.google.net", false},
+      {"http://192.168.1.2", false},
+
+      // Check both a bare eTLD.
+      {"https://sp.nom.br", false},
+  };
+
+  for (TestCase test_case : test_cases)
+    RunTestCase(test_case, filter);
+}
+
+TEST(BrowsingDataFilterBuilderTest, GURLBlacklist) {
+  BrowsingDataFilterBuilder builder(BrowsingDataFilterBuilder::BLACKLIST);
+  builder.AddRegisterableDomain(std::string(kGoogleDomain));
+  builder.AddRegisterableDomain(std::string(kLongETLDDomain));
+  builder.AddRegisterableDomain(std::string(kIPAddress));
+  base::Callback<bool(const GURL&)> filter = builder.BuildSameDomainFilter();
+
+  TestCase test_cases[] = {
+      // We matches any URL that are not on the specified domains.
+      {"http://www.google.com/foo/bar", false},
+      {"https://www.sub.google.com/foo/bar", false},
+      {"https://sub.google.com", false},
+      {"http://www.sub.google.com:8000/foo/bar", false},
+      {"https://website.sp.nom.br", false},
+      {"https://www.website.sp.nom.br", false},
+      {"http://192.168.1.1", false},
+      {"http://192.168.1.1:80", false},
+
+      // Different domains.
+      {"https://www.youtube.com", true},
+      {"https://www.google.net", true},
+      {"http://192.168.1.2", true},
+
+      // Check our bare eTLD.
+      {"https://sp.nom.br", true},
+  };
+
+  for (TestCase test_case : test_cases)
+    RunTestCase(test_case, filter);
+}
+
+TEST(BrowsingDataFilterBuilderTest, WhitelistContentSettings) {
+  BrowsingDataFilterBuilder builder(BrowsingDataFilterBuilder::WHITELIST);
+  builder.AddRegisterableDomain(std::string(kGoogleDomain));
+  builder.AddRegisterableDomain(std::string(kLongETLDDomain));
+  builder.AddRegisterableDomain(std::string(kIPAddress));
+  base::Callback<bool(const ContentSettingsPattern&)> filter =
+      builder.BuildWebsiteSettingsPatternMatchesFilter();
+
+  TestCase test_cases[] = {
+      // Whitelist matches any patterns that include the whitelisted
+      // registerable domains.
+      {"https://www.google.com", true},
+      {"http://www.google.com", true},
+      {"http://www.google.com/index.html", true},
+      {"http://www.google.com/foo/bar", true},
+      {"www.sub.google.com/bar", true},
+      {"http://www.sub.google.com:8000/foo/bar", true},
+      {"https://[*.]google.com", true},
+      {"https://[*.]google.com:443", true},
+      {"[*.]google.com", true},
+      {"[*.]google.com/foo/bar", true},
+      {"[*.]google.com:80", true},
+      {"www.google.com/?q=test", true},
+      {"192.168.1.1", true},
+      {"192.168.1.1:80", true},
+      // We check that we treat the eTLDs correctly.
+      {"https://website.sp.nom.br", true},
+      {"https://www.website.sp.nom.br", true},
+
+      // Different eTLDs, and a bare eTLD.
+      {"[*.]google", false},
+      {"[*.]google.net", false},
+      {"https://[*.]google.org", false},
+      {"https://[*.]foo.bar.com", false},
+      {"https://sp.nom.br", false},
+      {"http://192.168.1.2", false},
+
+      // These patterns are more general than our registerable domain filter,
+      // as they apply to more sites. So we don't match them. The content
+      // settings categories that we'll be seeing from browsing_data_remover
+      // aren't going to have these, but I test for it anyways.
+      {"*", false},
+      {"*:80", false}
+  };
+
+  for (TestCase test_case : test_cases)
+    RunTestCase(test_case, filter);
+}
+
+TEST(BrowsingDataFilterBuilderTest, BlacklistContentSettings) {
+  BrowsingDataFilterBuilder builder(BrowsingDataFilterBuilder::BLACKLIST);
+  builder.AddRegisterableDomain(std::string(kGoogleDomain));
+  builder.AddRegisterableDomain(std::string(kLongETLDDomain));
+  builder.AddRegisterableDomain(std::string(kIPAddress));
+  base::Callback<bool(const ContentSettingsPattern&)> filter =
+      builder.BuildWebsiteSettingsPatternMatchesFilter();
+
+  TestCase test_cases[] = {
+      // Blacklist matches any patterns that isn't in the blacklist
+      // registerable domains.
+      {"https://www.google.com", false},
+      {"http://www.google.com", false},
+      {"http://www.google.com/index.html", false},
+      {"http://www.google.com/foo/bar", false},
+      {"www.sub.google.com/bar", false},
+      {"http://www.sub.google.com:8000/foo/bar", false},
+      {"https://[*.]google.com", false},
+      {"https://[*.]google.com:443", false},
+      {"[*.]google.com", false},
+      {"[*.]google.com/foo/bar", false},
+      {"[*.]google.com:80", false},
+      {"www.google.com/?q=test", false},
+      {"192.168.1.1", false},
+      {"192.168.1.1:80", false},
+      // We check that we treat the eTLDs correctly.
+      {"https://website.sp.nom.br", false},
+      {"https://www.website.sp.nom.br", false},
+
+      // Different eTLDs, and a bare eTLD.
+      {"[*.]google", true},
+      {"[*.]google.net", true},
+      {"https://[*.]google.org", true},
+      {"https://[*.]foo.bar.com", true},
+      {"https://sp.nom.br", true},
+      {"http://192.168.1.2", true},
+
+      // These patterns are more general than our registerable domain filter,
+      // as they apply to more sites. So we don't match them. The content
+      // settings categories that we'll be seeing from browsing_data_remover
+      // aren't going to have these, but I test for it anyways.
+      {"*", true},
+      {"*:80", true}
+  };
+
+  for (TestCase test_case : test_cases)
+    RunTestCase(test_case, filter);
+}
+
+TEST(BrowsingDataFilterBuilderTest, MatchesCookiesWhitelist) {
+  BrowsingDataFilterBuilder builder(BrowsingDataFilterBuilder::WHITELIST);
+  builder.AddRegisterableDomain(std::string(kGoogleDomain));
+  builder.AddRegisterableDomain(std::string(kLongETLDDomain));
+  builder.AddRegisterableDomain(std::string(kIPAddress));
+  base::Callback<bool(const net::CanonicalCookie&)> filter =
+      builder.BuildDomainCookieFilter();
+
+  TestCase test_cases[] = {
+      // Any cookie with the same registerable domain as the origins is matched.
+      {"https://www.google.com", true},
+      {"http://www.google.com", true},
+      {"http://www.google.com:300", true},
+      {"https://mail.google.com", true},
+      {"http://mail.google.com", true},
+      {"http://google.com", true},
+      {"https://website.sp.nom.br", true},
+      {"https://sub.website.sp.nom.br", true},
+      {"http://192.168.1.1", true},
+      {"http://192.168.1.1:10", true},
+
+      // Different eTLDs.
+      {"https://www.google.org", false},
+      {"https://www.google.co.uk", false},
+
+      // We treat eTLD+1 and bare eTLDs as different domains.
+      {"https://www.sp.nom.br", false},
+      {"https://sp.nom.br", false},
+
+      // Different hosts in general.
+      {"https://www.chrome.com", false},
+      {"http://192.168.2.1", false}};
+
+  for (TestCase test_case : test_cases)
+    RunTestCase(test_case, filter);
+}
+
+TEST(BrowsingDataFilterBuilderTest, MatchesCookiesBlacklist) {
+  BrowsingDataFilterBuilder builder(BrowsingDataFilterBuilder::BLACKLIST);
+  builder.AddRegisterableDomain(std::string(kGoogleDomain));
+  builder.AddRegisterableDomain(std::string(kLongETLDDomain));
+  builder.AddRegisterableDomain(std::string(kIPAddress));
+  base::Callback<bool(const net::CanonicalCookie&)> filter =
+      builder.BuildDomainCookieFilter();
+
+  TestCase test_cases[] = {
+      // Any cookie that doesn't have the same registerable domain is matched.
+      {"https://www.google.com", false},
+      {"http://www.google.com", false},
+      {"http://www.google.com:300", false},
+      {"https://mail.google.com", false},
+      {"http://mail.google.com", false},
+      {"http://google.com", false},
+      {"https://website.sp.nom.br", false},
+      {"https://sub.website.sp.nom.br", false},
+      {"http://192.168.1.1", false},
+      {"http://192.168.1.1:10", false},
+
+      // Different eTLDs.
+      {"https://www.google.org", true},
+      {"https://www.google.co.uk", true},
+
+      // We treat eTLD+1 and bare eTLDs as different domains.
+      {"https://www.sp.nom.br", true},
+      {"https://sp.nom.br", true},
+
+      // Different hosts in general.
+      {"https://www.chrome.com", true},
+      {"http://192.168.2.1", true}};
+
+  for (TestCase test_case : test_cases)
+    RunTestCase(test_case, filter);
+}