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

Unified Diff: components/update_client/ping_manager.cc

Issue 2340113002: Inject the prodid for the extensions and component updaters. (Closed)
Patch Set: Fix iOS broken build. Created 4 years, 3 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/update_client/ping_manager.cc
diff --git a/components/update_client/ping_manager.cc b/components/update_client/ping_manager.cc
index 59ee87523e340abf9042829c33c1b7adaaa4010b..47ae77c3cf4d1a1ebe09c37e3584dc345fdee5e1 100644
--- a/components/update_client/ping_manager.cc
+++ b/components/update_client/ping_manager.cc
@@ -162,10 +162,10 @@ std::string BuildPing(const Configurator& config, const CrxUpdateItem* item) {
ping_event.c_str(), // ping event
BuildDownloadCompleteEventElements(item).c_str())); // download events
- return BuildProtocolRequest(config.GetBrowserVersion().GetString(),
- config.GetChannel(), config.GetLang(),
- config.GetOSLongName(),
- config.GetDownloadPreference(), app_element, "");
+ return BuildProtocolRequest(
+ config.GetProdId(), config.GetBrowserVersion().GetString(),
+ config.GetChannel(), config.GetLang(), config.GetOSLongName(),
+ config.GetDownloadPreference(), app_element, "");
}
// Sends a fire and forget ping. The instances of this class have no

Powered by Google App Engine
This is Rietveld 408576698