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

Unified Diff: chrome/browser/component_updater/component_updater_utils.cc

Issue 334783002: Componentize component_updater: Move some paths/constants to component. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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: chrome/browser/component_updater/component_updater_utils.cc
diff --git a/chrome/browser/component_updater/component_updater_utils.cc b/chrome/browser/component_updater/component_updater_utils.cc
index ec6d497ac329fb6931331ece0c8b9d7bd091207c..6e45ea99734ff63fba0ff22773c37d76d8531dc6 100644
--- a/chrome/browser/component_updater/component_updater_utils.cc
+++ b/chrome/browser/component_updater/component_updater_utils.cc
@@ -14,11 +14,11 @@
#include "base/strings/string_util.h"
#include "base/strings/stringprintf.h"
#include "base/sys_info.h"
+#include "base/version.h"
#include "base/win/windows_version.h"
#include "chrome/browser/component_updater/component_updater_service.h"
#include "chrome/browser/component_updater/crx_update_item.h"
#include "chrome/browser/omaha_query_params/omaha_query_params.h"
-#include "chrome/common/chrome_version_info.h"
#include "extensions/common/extension.h"
#include "net/base/load_flags.h"
#include "net/url_request/url_fetcher.h"
@@ -38,12 +38,12 @@ int GetPhysicalMemoryGB() {
} // namespace
-std::string BuildProtocolRequest(const std::string& request_body,
+std::string BuildProtocolRequest(const std::string& chrome_version,
+ const std::string& platform_name,
+ const std::string& request_body,
const std::string& additional_attributes) {
const std::string prod_id(chrome::OmahaQueryParams::GetProdIdString(
chrome::OmahaQueryParams::CHROME));
- const chrome::VersionInfo chrome_version_info;
- const std::string chrome_version(chrome_version_info.Version());
std::string request(
"<?xml version=\"1.0\" encoding=\"UTF-8\"?>"
@@ -85,7 +85,7 @@ std::string BuildProtocolRequest(const std::string& request_body,
base::StringAppendF(
&request,
"<os platform=\"%s\" version=\"%s\" arch=\"%s\"/>",
- chrome::VersionInfo().OSType().c_str(), // "platform"
+ platform_name.c_str(), // "platform"
base::SysInfo().OperatingSystemVersion().c_str(), // "version"
base::SysInfo().OperatingSystemArchitecture().c_str()); // "arch"

Powered by Google App Engine
This is Rietveld 408576698