diff options
| author | justincohen <justincohen@chromium.org> | 2015-04-07 19:14:28 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-04-08 02:15:09 +0000 |
| commit | c438c1843041746877c9dc83d9ce63a3c43bb6bb (patch) | |
| tree | 650acf54236e317ba78fc7d916dae0d4fa0da923 /chrome/browser/android/logo_service.cc | |
| parent | 5729f982048935b2b702181d0b349589d607f34b (diff) | |
| download | chromium_src-c438c1843041746877c9dc83d9ce63a3c43bb6bb.zip chromium_src-c438c1843041746877c9dc83d9ce63a3c43bb6bb.tar.gz chromium_src-c438c1843041746877c9dc83d9ce63a3c43bb6bb.tar.bz2 | |
[LogoTracker] Correct fingerprint append in async parameter.
Clients that already set the async parameter (in this case, with 'cta:1'), will
end up with query strings with two differeny async parameters, instead of a
comma delimited async param.
E.g. converts ?async=cta:1&async=es_dfp:123 -> ?async=cta:1,es_dfp:123
BUG=414528
Review URL: https://codereview.chromium.org/1066923002
Cr-Commit-Position: refs/heads/master@{#324174}
Diffstat (limited to 'chrome/browser/android/logo_service.cc')
| -rw-r--r-- | chrome/browser/android/logo_service.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/android/logo_service.cc b/chrome/browser/android/logo_service.cc index 17ec968..c71cefd 100644 --- a/chrome/browser/android/logo_service.cc +++ b/chrome/browser/android/logo_service.cc @@ -124,7 +124,8 @@ void LogoService::GetLogo(search_provider_logos::LogoObserver* observer) { logo_tracker_->SetServerAPI( GetGoogleDoodleURL(profile_), base::Bind(&search_provider_logos::GoogleParseLogoResponse), - base::Bind(&search_provider_logos::GoogleAppendFingerprintToLogoURL)); + base::Bind(&search_provider_logos::GoogleAppendQueryparamsToLogoURL), + false); logo_tracker_->GetLogo(observer); } |