[Prototype] Delete Google service worker caches on Android signout

components/google/core/browser/google_util.cc

 // Copyright 2014 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 "components/google/core/browser/google_util.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/google/core/browser/google_switches.h"
 #include "components/google/core/browser/google_tld_list.h"
 #include "components/google/core/browser/google_url_tracker.h"
 #include "components/url_formatter/url_fixer.h"
 #include "net/base/registry_controlled_domains/registry_controlled_domain.h"
 #include "net/base/url_util.h"
 #include "url/gurl.h"
 
 // Only use Link Doctor on official builds.  It uses an API key, too, but
@@ skipping 248 matching lines @@
 }
 
 bool IsYoutubeDomainUrl(const GURL& url,
                         SubdomainPermission subdomain_permission,
                         PortPermission port_permission) {
   return IsValidURL(url, port_permission) &&
       IsValidHostName(url.host_piece(), "youtube", subdomain_permission,
                       nullptr);
 }
 
+const std::vector<std::string>& GetGoogleRegisterableDomains() {
+  CR_DEFINE_STATIC_LOCAL(std::vector<std::string>, kGoogleRegisterableDomains,
+                         ());
+
+  // Initialize the list.
+  if (kGoogleRegisterableDomains.empty()) {
+    std::vector<std::string> tlds({GOOGLE_TLD_LIST});

[Peter Kasting, 2017/07/10 19:26:05]

Nit: Are () necessary here?

[msramek, 2017/07/13 14:19:47]

Done. No, and I actually didn't know this was possible :)

+    for (const std::string& tld : tlds) {
+      std::string domain = base::StringPrintf("google.%s", tld.c_str());

[Peter Kasting, 2017/07/10 19:26:05]

Nit: Easier to read, doesn't require an #include, probably more performant:

  std::string domain = "google." + tld;

[msramek, 2017/07/13 14:19:47]

Done.

+
+      // Our list of domains may be slightly out of date.

[Peter Kasting, 2017/07/10 19:26:05]

I kinda don't think you should do this check.  It's not clear to me exactly what
sort of out-of-dateness it's supposed to solve, especially since the items on
GOOGLE_TLD_LIST should be using ccTLDs.  If a ccTLD went away, I think this
check wouldn't fail?  Anyway, if we wanted to keep this, I'd rather it be more
clearly motivated.

Also, without this, this whole loop basically just becomes a call to
std::transform with a back_inserter, or even maybe you just set
kGoogleRegisterableDomains = {GOOGLE_TLD_LIST} and then transform in-place.

[msramek, 2017/07/13 14:19:47]

Sorry, the comment was unhelpful, because I wrote it before I fully investigated
the problem.

The browsing_data/ machinery for deleting registrable domains has a DCHECK()
that all inputs are really registrable domains.

...and this DCHECK() fired. Turns out that the TLD list, generated from our DNS
records, contained something that was not actually a ccTLD.

See here:
https://bugs.chromium.org/p/chromium/issues/detail?id=674712#c14
and here:
b/63446999

This particular case can be resolved, but I wanted to avoid hardcoding an
assumption that the two lists are in sync for future's sake.

+      if (GetDomainAndRegistry(
+              domain,
+              net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES) !=
+          domain) {
+        continue;
+      }
+
+      kGoogleRegisterableDomains.push_back(domain);
+    }
+  }
+
+  return kGoogleRegisterableDomains;
+}
+
 }  // namespace google_util