Introduce Animated Logo to Chrome on Android

components/search_provider_logos/logo_tracker.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_SEARCH_PROVIDER_LOGOS_LOGO_TRACKER_H_
 #define COMPONENTS_SEARCH_PROVIDER_LOGOS_LOGO_TRACKER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 54 matching lines @@
 };
 
 // Parses the response from the server and returns it as an EncodedLogo. Returns
 // NULL if the response is invalid.
 typedef base::Callback<
     scoped_ptr<EncodedLogo>(const scoped_ptr<std::string>& response,
                             base::Time response_time)> ParseLogoResponse;
 
 // Encodes the fingerprint of the cached logo in the logo URL. This enables the
 // server to verify whether the cached logo is up-to-date.
+// TODO(ianwen): remove wants_cta from parameter.

[newt (away), 2015/09/23 20:38:44]

I approve of this TODO, but it's in the wrong place.

[Ian Wen, 2015/09/23 21:59:52]

Done.

 typedef base::Callback<GURL(const GURL& logo_url,
                             const std::string& fingerprint,
                             bool wants_cta)> AppendQueryparamsToLogoURL;
 
 // This class provides the logo for a search provider. Logos are downloaded from
 // the search provider's logo URL and cached on disk.
 //
 // Call SetServerAPI() at least once to specify how to get the logo from the
 // server. Then call GetLogo() to trigger retrieval of the logo and receive
 // updates once the cached and/or fresh logos are available.
@@ skipping 144 matching lines @@
   scoped_refptr<net::URLRequestContextGetter> request_context_getter_;
 
   base::WeakPtrFactory<LogoTracker> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(LogoTracker);
 };
 
 }  // namespace search_provider_logos
 
 #endif  // COMPONENTS_SEARCH_PROVIDER_LOGOS_LOGO_TRACKER_H_