diff options
| author | gcasto@chromium.org <gcasto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-08 00:13:11 +0000 |
|---|---|---|
| committer | gcasto@chromium.org <gcasto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-08 00:13:11 +0000 |
| commit | c35e533db8c4331a4c390afe8b2ff2487f227122 (patch) | |
| tree | e36f5c57c22b1e10d764d370b23ea389700fb629 /chrome/browser/safe_browsing/client_side_detection_service_unittest.cc | |
| parent | 67af4652c9fa25b7adb09fd3dbb6207cd78b4e3b (diff) | |
| download | chromium_src-c35e533db8c4331a4c390afe8b2ff2487f227122.zip chromium_src-c35e533db8c4331a4c390afe8b2ff2487f227122.tar.gz chromium_src-c35e533db8c4331a4c390afe8b2ff2487f227122.tar.bz2 | |
Change browser features to use redirect chain for redirects tracking.
Previous use of masks in the page transition type are broken, as the masks are
not always set before the entries are placed in the history backend.
BUG=
TEST=Ran unittests for BrowserFeatureExtraction, ClientSideDetectionHost, and ClientSideDetectionService
Review URL: http://codereview.chromium.org/8073004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104613 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/safe_browsing/client_side_detection_service_unittest.cc')
| -rw-r--r-- | chrome/browser/safe_browsing/client_side_detection_service_unittest.cc | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/chrome/browser/safe_browsing/client_side_detection_service_unittest.cc b/chrome/browser/safe_browsing/client_side_detection_service_unittest.cc index ba0a5a7..fa09a39 100644 --- a/chrome/browser/safe_browsing/client_side_detection_service_unittest.cc +++ b/chrome/browser/safe_browsing/client_side_detection_service_unittest.cc @@ -683,17 +683,17 @@ TEST_F(ClientSideDetectionServiceTest, SanitizeRequestForPingback) { AddNonModelFeature(StringPrintf("%s=http://referrer.com/", features::kReferrer), 1.0, &request); - AddNonModelFeature(StringPrintf("%s%s=http://redirreferrer.com/", - features::kRedirectPrefix, - features::kReferrer), + AddNonModelFeature(StringPrintf("%s[0]=%s", + features::kRedirect, + "http://www.redirect.com/"), 1.0, &request); AddNonModelFeature(StringPrintf("%s%s=http://hostreferrer.com/", features::kHostPrefix, features::kReferrer), 1.0, &request); - AddNonModelFeature(StringPrintf("%s%s%s=http://hostredirreferrer.com/", + AddNonModelFeature(StringPrintf("%s%s[1]=%s", features::kHostPrefix, - features::kRedirectPrefix, - features::kReferrer), + features::kRedirect, + "http://www.other_redirect.com/"), 1.0, &request); AddNonModelFeature(std::string(features::kBadIpFetch) + "1.2.3.4", 1.0, &request); |