Chromium Code Reviews| Index: components/domain_reliability/google_configs.cc |
| diff --git a/components/domain_reliability/google_configs.cc b/components/domain_reliability/google_configs.cc |
| index 740673d699c18e8e11d0b75d2ea18e40da7a78d9..99bdcc4d2b8741a823cbdf6a7924d922756edb0a 100644 |
| --- a/components/domain_reliability/google_configs.cc |
| +++ b/components/domain_reliability/google_configs.cc |
| @@ -31,7 +31,9 @@ struct GoogleConfigParams { |
| }; |
| 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 }, |
| @@ -88,7 +90,6 @@ const GoogleConfigParams kGoogleConfigs[] = { |
| { "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 }, |
| @@ -253,6 +254,9 @@ const GoogleConfigParams kGoogleConfigs[] = { |
| { "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 go
|
| + // 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 }, |
| @@ -292,9 +296,9 @@ const GoogleConfigParams kGoogleConfigs[] = { |
| { "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 }, |
| @@ -317,8 +321,8 @@ const GoogleConfigParams kGoogleConfigs[] = { |
| { "google.info", true, false, false }, |
| { "google.jobs", true, false, false }, |
| { "google.net", true, false, false }, |
| - { "google.org", 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 }, |
| { "googleadapis.com", true, false, false }, |
| { "googleadservices.com", true, false, false }, |
| { "googleadsserving.cn", true, false, false }, |
| @@ -343,6 +347,8 @@ const GoogleConfigParams kGoogleConfigs[] = { |
| { "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 }, |
| @@ -498,14 +504,14 @@ const GoogleConfigParams kGoogleConfigs[] = { |
| // 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 }, |