Componentize component_updater: Move some paths/constants to component.

chrome/browser/component_updater/update_checker.cc

Index: chrome/browser/component_updater/update_checker.cc
diff --git a/chrome/browser/component_updater/update_checker.cc b/chrome/browser/component_updater/update_checker.cc
index 061671aa6a999db22039c9213b6d8728987a6a7e..c58ce2d98d1fe04cec19924403908b7e6047f08c 100644
--- a/chrome/browser/component_updater/update_checker.cc
+++ b/chrome/browser/component_updater/update_checker.cc
@@ -32,7 +32,9 @@ namespace component_updater {
 //        <package fp="abcd" />
 //      </packages>
 //    </app>
-std::string BuildUpdateCheckRequest(const std::vector<CrxUpdateItem*>& items,
+std::string BuildUpdateCheckRequest(const std::string& application_version,
+                                    const std::string& platform_name,
+                                    const std::vector<CrxUpdateItem*>& items,
                                     const std::string& additional_attributes) {
   std::string app_elements;
   for (size_t i = 0; i != items.size(); ++i) {
@@ -58,7 +60,8 @@ std::string BuildUpdateCheckRequest(const std::vector<CrxUpdateItem*>& items,
     VLOG(1) << "Appending to update request: " << app;
   }
 
-  return BuildProtocolRequest(app_elements, additional_attributes);
+  return BuildProtocolRequest(
+      application_version, platform_name, app_elements, additional_attributes);

[Sorin Jianu, 2014/06/19 00:48:22]

Same thing as for the ping manager.

 }
 
 class UpdateCheckerImpl : public UpdateChecker, public net::URLFetcherDelegate {
@@ -70,6 +73,8 @@ class UpdateCheckerImpl : public UpdateChecker, public net::URLFetcherDelegate {
 
   // Overrides for UpdateChecker.
   virtual bool CheckForUpdates(
+      const std::string& application_version,
+      const std::string& platform_name,
       const std::vector<CrxUpdateItem*>& items_to_check,
       const std::string& additional_attributes) OVERRIDE;
 
@@ -110,6 +115,8 @@ UpdateCheckerImpl::~UpdateCheckerImpl() {
 }
 
 bool UpdateCheckerImpl::CheckForUpdates(
+    const std::string& application_version,
+    const std::string& platform_name,
     const std::vector<CrxUpdateItem*>& items_to_check,
     const std::string& additional_attributes) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
@@ -117,11 +124,14 @@ bool UpdateCheckerImpl::CheckForUpdates(
   if (url_fetcher_)
     return false;  // Another fetch is in progress.
 
-  url_fetcher_.reset(SendProtocolRequest(
-      url_,
-      BuildUpdateCheckRequest(items_to_check, additional_attributes),
-      this,
-      url_request_context_getter_));
+  url_fetcher_.reset(
+      SendProtocolRequest(url_,
+                          BuildUpdateCheckRequest(application_version,
+                                                  platform_name,
+                                                  items_to_check,
+                                                  additional_attributes),
+                          this,
+                          url_request_context_getter_));
 
   return true;
 }