Domain Reliability: Update Google domain list.

components/domain_reliability/google_configs.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 "components/domain_reliability/google_configs.h"
 
 #include <stddef.h>
 
 #include <memory>
 
@@ skipping 13 matching lines @@
 
   // If true, also add a config for www.|hostname|.
   //
   // |include_subdomains| will be false in the extra config, but
   // |include_origin_specific_collector| will be respected, and will use the
   // www subdomain as the origin for the collector so it matches the config.
   bool duplicate_for_www;
 };
 
 const GoogleConfigParams kGoogleConfigs[] = {
-  // Origins with subdomains and same-origin collectors.
+  // Origins with subdomains and same-origin collectors. Currently, all
+  // origins with same-origin collectors also run collectors on their www
+  // subdomain. (e.g., both foo.com and www.foo.com.)
   { "google.ac", true, true, true },
   { "google.ad", true, true, true },
   { "google.ae", true, true, true },
   { "google.af", true, true, true },
   { "google.ag", true, true, true },
   { "google.al", true, true, true },
   { "google.am", true, true, true },
   { "google.as", true, true, true },
   { "google.at", true, true, true },
   { "google.az", true, true, true },
@@ skipping 36 matching lines @@
   { "google.co.th", true, true, true },
   { "google.co.tz", true, true, true },
   { "google.co.ug", true, true, true },
   { "google.co.uk", true, true, true },
   { "google.co.uz", true, true, true },
   { "google.co.ve", true, true, true },
   { "google.co.vi", true, true, true },
   { "google.co.za", true, true, true },
   { "google.co.zm", true, true, true },
   { "google.co.zw", true, true, true },
-  { "google.com", true, true, false },
   { "google.com.af", true, true, true },
   { "google.com.ag", true, true, true },
   { "google.com.ai", true, true, true },
   { "google.com.ar", true, true, true },
   { "google.com.au", true, true, true },
   { "google.com.bd", true, true, true },
   { "google.com.bh", true, true, true },
   { "google.com.bn", true, true, true },
   { "google.com.bo", true, true, true },
   { "google.com.br", true, true, true },
@@ skipping 144 matching lines @@
   { "google.tm", true, true, true },
   { "google.tn", true, true, true },
   { "google.to", true, true, true },
   { "google.tt", true, true, true },
   { "google.us", true, true, true },
   { "google.uz", true, true, true },
   { "google.vg", true, true, true },
   { "google.vu", true, true, true },
   { "google.ws", true, true, true },
   { "l.google.com", true, true, true },
+  // google.com is a special case. We have a custom config for www.google.com,

[cbentzel, 2016/08/24 14:11:44]

Nit: I'd add a newline between l.google.com and google.com as this is
effectively a different section. Would also explain why not lexicographically
sorted.

+  // so set generate_config_for_www_subdomain = false.
+  { "google.com", true, true, false },
 
   // Origins with subdomains and without same-origin collectors.
   { "2mdn.net", true, false, false },
   { "adgoogle.net", true, false, false },
   { "admeld.com", true, false, false },
   { "admob.biz", true, false, false },
   { "admob.co.in", true, false, false },
   { "admob.co.kr", true, false, false },
   { "admob.co.nz", true, false, false },
   { "admob.co.uk", true, false, false },
@@ skipping 19 matching lines @@
   { "admob.mobi", true, false, false },
   { "admob.no", true, false, false },
   { "admob.ph", true, false, false },
   { "admob.pt", true, false, false },
   { "admob.sg", true, false, false },
   { "admob.tw", true, false, false },
   { "admob.us", true, false, false },
   { "admob.vn", true, false, false },
   { "adwhirl.com", true, false, false },
   { "android.com", true, false, false },
+  { "anycast.metric.gstatic.com", true, false, false },
   { "anycast-edge.metric.gstatic.com", true, false, false },
-  { "anycast1.metric.gstatic.com", true, false, false },
-  { "anycast1-stb.metric.gstatic.com", true, false, false },
+  { "anycast-stb.metric.gstatic.com", true, false, false },
   { "chromecast.com", true, false, false },
   { "chromeexperiments.com", true, false, false },
   { "chromestatus.com", true, false, false },
   { "chromium.org", true, false, false },
   { "cloudendpointsapis.com", true, false, false },
   { "dartmotif.com", true, false, false },
   { "dartsearch.net", true, false, false },
   { "doubleclick.com", true, false, false },
   { "doubleclick.ne.jp", true, false, false },
   { "doubleclick.net", true, false, false },
   { "doubleclickusercontent.com", true, false, false },
   { "fls.doubleclick.net", true, false, false },
   { "g.co", true, false, false },
   { "g.doubleclick.net", true, false, false },
   { "ggpht.com", true, false, false },
   { "gmodules.com", true, false, false },
   { "goo.gl", true, false, false },
   { "google-syndication.com", true, false, false },
   { "google.cat", true, false, false },
   { "google.info", true, false, false },
   { "google.jobs", true, false, false },
   { "google.net", true, false, false },
+  { "google.stackdriver.com", true, false, false },

[cbentzel, 2016/08/24 14:11:44]

Nit: this seems out of order

   { "google.org", true, false, false },
-  { "google.stackdriver.com", true, false, false },
   { "googleadapis.com", true, false, false },
   { "googleadservices.com", true, false, false },
   { "googleadsserving.cn", true, false, false },
   { "googlealumni.com", true, false, false },
   { "googleapis.cn", true, false, false },
   { "googleapis.com", true, false, false },
   { "googleapps.com", true, false, false },
   { "googlecbs.com", true, false, false },
   { "googlecode.com", true, false, false },
   { "googlecommerce.com", true, false, false },
   { "googledrive.com", true, false, false },
   { "googleenterprise.com", true, false, false },
   { "googlefiber.com", true, false, false },
   { "googlefiber.net", true, false, false },
   { "googlegoro.com", true, false, false },
   { "googlehosted.com", true, false, false },
   { "googlepayments.com", true, false, false },
   { "googlesource.com", true, false, false },
   { "googlesyndication.com", true, false, false },
   { "googletagmanager.com", true, false, false },
   { "googletagservices.com", true, false, false },
   { "googleusercontent.com", true, false, false },
   { "gstatic.cn", true, false, false },
   { "gstatic.com", true, false, false },
+  { "gvt3.com", true, false, false },

[cbentzel, 2016/08/24 14:11:44]

What are these?

+  { "gvt9.com", true, false, false },
   { "picasa.com", true, false, false },
   { "recaptcha.net", true, false, false },
   { "stackdriver.com", true, false, false },
   { "stbcast.metric.gstatic.com", true, false, false },
   { "stbcast2.metric.gstatic.com", true, false, false },
   { "stbcast3.metric.gstatic.com", true, false, false },
   { "stbcast4.metric.gstatic.com", true, false, false },
   { "stbcast-stb.metric.gstatic.com", true, false, false },
   { "stbcast2-stb.metric.gstatic.com", true, false, false },
   { "stbcast3-stb.metric.gstatic.com", true, false, false },
@@ skipping 135 matching lines @@
   { "youtube.ug", true, false, false },
   { "youtube.uy", true, false, false },
   { "youtube.vn", true, false, false },
   { "youtubeeducation.com", true, false, false },
   { "youtubemobilesupport.com", true, false, false },
   { "ytimg.com", true, false, false },
 
   // Origins without subdomains and with same-origin collectors.
   { "accounts.google.com", false, true, false },
   { "apis.google.com", false, true, false },
+  { "ddm.google.com", false, true, false },
+  { "mail.google.com", false, true, false },
   { "b.mail.google.com", false, true, false },
   { "chatenabled.mail.google.com", false, true, false },
-  { "ddm.google.com", false, true, false },
+  { "www.gmail.com", false, true, false },
   { "gmail.com", false, true, false },
   { "gmail.google.com", false, true, false },
-  { "mail.google.com", false, true, false },
   { "mail-attachment.googleusercontent.com", false, true, false },
-  { "www.gmail.com", false, true, false },
 
   // Origins without subdomains or same-origin collectors.
   { "ad.doubleclick.net", false, false, false },
   { "drive.google.com", false, false, false },
   { "redirector.googlevideo.com", false, false, false },
 };
 
 const char* kGoogleStandardCollectors[] = {
   "https://beacons.gvt2.com/domainreliability/upload",
   "https://beacons2.gvt2.com/domainreliability/upload",
@@ skipping 43 matching lines @@
   configs_out->clear();
 
   for (auto& params : kGoogleConfigs) {
     configs_out->push_back(CreateGoogleConfig(params, false).release());
     if (params.duplicate_for_www)
       configs_out->push_back(CreateGoogleConfig(params, true).release());
   }
 }
 
 }  // namespace domain_reliability