Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4232)

Unified Diff: chrome/browser/metrics/variations/variations_http_header_provider_unittest.cc

Issue 224323002: Merge 260929 "Send Finch experiment selection data to ad-service..." (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1916/src/
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/metrics/variations/variations_http_header_provider.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/variations/variations_http_header_provider_unittest.cc
===================================================================
--- chrome/browser/metrics/variations/variations_http_header_provider_unittest.cc (revision 261467)
+++ chrome/browser/metrics/variations/variations_http_header_provider_unittest.cc (working copy)
@@ -46,14 +46,38 @@
{ "http://google.co.uk", true },
{ "http://google.co.uk:8080/", true },
{ "http://www.google.co.uk:8080/", true },
+ { "http://google", false },
+
{ "http://youtube.com", true },
{ "http://www.youtube.com", true },
{ "http://www.youtube.ca", true },
{ "http://www.youtube.co.uk:8080/", true },
{ "https://www.youtube.com", true },
+ { "http://youtube", false },
+
{ "http://www.yahoo.com", false },
- { "http://youtube", false },
- { "http://google", false },
+
+ { "http://ad.doubleclick.net", true },
+ { "https://a.b.c.doubleclick.net", true },
+ { "https://a.b.c.doubleclick.net:8081", true },
+ { "http://www.doubleclick.com", false },
+ { "http://www.doubleclick.net.com", false },
+ { "https://www.doubleclick.net.com", false },
+
+ { "http://ad.googlesyndication.com", true },
+ { "https://a.b.c.googlesyndication.com", true },
+ { "https://a.b.c.googlesyndication.com:8080", true },
+ { "http://www.doubleclick.edu", false },
+ { "http://www.googlesyndication.com.edu", false },
+ { "https://www.googlesyndication.com.com", false },
+
+ { "http://www.googleadservices.com", true },
+ { "http://www.googleadservices.com:8080", true },
+ { "https://www.googleadservices.com", true },
+ { "https://www.internal.googleadservices.com", false },
+ { "https://www2.googleadservices.com", false },
+ { "https://www.googleadservices.org", false },
+ { "https://www.googleadservices.com.co.uk", false },
};
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) {
« no previous file with comments | « chrome/browser/metrics/variations/variations_http_header_provider.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698