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 /components/search_provider_logos/google_logo_api.h | |
| 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 'components/search_provider_logos/google_logo_api.h')
| -rw-r--r-- | components/search_provider_logos/google_logo_api.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/components/search_provider_logos/google_logo_api.h b/components/search_provider_logos/google_logo_api.h index 569752d..84e4194 100644 --- a/components/search_provider_logos/google_logo_api.h +++ b/components/search_provider_logos/google_logo_api.h @@ -16,8 +16,9 @@ namespace search_provider_logos { // Implements AppendFingerprintToLogoURL, defined in logo_tracker.h, for Google // doodles. -GURL GoogleAppendFingerprintToLogoURL(const GURL& logo_url, - const std::string& fingerprint); +GURL GoogleAppendQueryparamsToLogoURL(const GURL& logo_url, + const std::string& fingerprint, + bool wants_cta); // Implements ParseLogoResponse, defined in logo_tracker.h, for Google doodles. scoped_ptr<EncodedLogo> GoogleParseLogoResponse( |