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

Unified Diff: components/search_provider_logos/logo_tracker.cc

Issue 1962013002: Adding a parameter that will be used to request a transparent doodle. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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
Index: components/search_provider_logos/logo_tracker.cc
diff --git a/components/search_provider_logos/logo_tracker.cc b/components/search_provider_logos/logo_tracker.cc
index bf2759a7e61024f6150c7764135cb9bf66957873..56bd0cac72993243255fecfdb691e08ec57b85c1 100644
--- a/components/search_provider_logos/logo_tracker.cc
+++ b/components/search_provider_logos/logo_tracker.cc
@@ -92,7 +92,8 @@ void LogoTracker::SetServerAPI(
const GURL& logo_url,
const ParseLogoResponse& parse_logo_response_func,
const AppendQueryparamsToLogoURL& append_queryparams_func,
- bool wants_cta) {
+ bool wants_cta,
+ bool transparent) {
if (logo_url == logo_url_)
return;
@@ -102,6 +103,7 @@ void LogoTracker::SetServerAPI(
parse_logo_response_func_ = parse_logo_response_func;
append_queryparams_func_ = append_queryparams_func;
wants_cta_ = wants_cta;
+ transparent_ = transparent;
}
void LogoTracker::GetLogo(LogoObserver* observer) {
@@ -212,7 +214,8 @@ void LogoTracker::FetchLogo() {
cached_logo_->metadata.expiration_time >= clock_->Now()) {
fingerprint = cached_logo_->metadata.fingerprint;
}
- url = append_queryparams_func_.Run(logo_url_, fingerprint, wants_cta_);
+ url = append_queryparams_func_.Run(
+ logo_url_, fingerprint, wants_cta_, transparent_);
fetcher_ = net::URLFetcher::Create(url, net::URLFetcher::GET, this);
fetcher_->SetRequestContext(request_context_getter_.get());
« no previous file with comments | « components/search_provider_logos/logo_tracker.h ('k') | components/search_provider_logos/logo_tracker_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698