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

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

Issue 333353005: Move OmahaQueryParams to a component and add a delegate interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase 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 bd23865154be61d0a6236199b6c4552f872dfabc..d7ff01b49960232503667b7b5248563b40140865 100644
--- a/chrome/browser/component_updater/component_updater_utils.cc
+++ b/chrome/browser/component_updater/component_updater_utils.cc
@@ -16,14 +16,17 @@
#include "base/sys_info.h"
#include "base/win/windows_version.h"
#include "chrome/browser/component_updater/crx_update_item.h"
-#include "chrome/browser/omaha_query_params/omaha_query_params.h"
+#include "chrome/browser/omaha_query_params/chrome_omaha_query_params_delegate.h"
#include "chrome/common/chrome_version_info.h"
+#include "components/omaha_query_params/omaha_query_params.h"
#include "extensions/common/extension.h"
#include "net/base/load_flags.h"
#include "net/url_request/url_fetcher.h"
#include "net/url_request/url_request_context_getter.h"
#include "net/url_request/url_request_status.h"
+using omaha_query_params::OmahaQueryParams;
+
namespace component_updater {
namespace {
@@ -39,10 +42,12 @@ int GetPhysicalMemoryGB() {
std::string BuildProtocolRequest(const std::string& request_body,
const std::string& additional_attributes) {
- const std::string prod_id(chrome::OmahaQueryParams::GetProdIdString(
- chrome::OmahaQueryParams::CHROME));
+ const std::string prod_id(
+ OmahaQueryParams::GetProdIdString(OmahaQueryParams::CHROME));
const chrome::VersionInfo chrome_version_info;
const std::string chrome_version(chrome_version_info.Version());
+ const std::string channel(ChromeOmahaQueryParamsDelegate::GetChannelString());
+ const std::string lang(ChromeOmahaQueryParamsDelegate::GetLang());
std::string request(
"<?xml version=\"1.0\" encoding=\"UTF-8\"?>"
@@ -58,15 +63,15 @@ std::string BuildProtocolRequest(const std::string& request_body,
"requestid=\"{%s}\" lang=\"%s\" updaterchannel=\"%s\" prodchannel=\"%s\" "
"os=\"%s\" arch=\"%s\" nacl_arch=\"%s\"",
prod_id.c_str(),
- chrome_version.c_str(), // "version"
- chrome_version.c_str(), // "prodversion"
- base::GenerateGUID().c_str(), // "requestid"
- chrome::OmahaQueryParams::GetLang(), // "lang",
- chrome::OmahaQueryParams::GetChannelString(), // "updaterchannel"
- chrome::OmahaQueryParams::GetChannelString(), // "prodchannel"
- chrome::OmahaQueryParams::GetOS(), // "os"
- chrome::OmahaQueryParams::GetArch(), // "arch"
- chrome::OmahaQueryParams::GetNaclArch()); // "nacl_arch"
+ chrome_version.c_str(), // "version"
+ chrome_version.c_str(), // "prodversion"
+ base::GenerateGUID().c_str(), // "requestid"
+ lang.c_str(), // "lang",
+ channel.c_str(), // "updaterchannel"
+ channel.c_str(), // "prodchannel"
+ OmahaQueryParams::GetOS(), // "os"
+ OmahaQueryParams::GetArch(), // "arch"
+ OmahaQueryParams::GetNaclArch()); // "nacl_arch"
#if defined(OS_WIN)
const bool is_wow64(base::win::OSInfo::GetInstance()->wow64_status() ==
base::win::OSInfo::WOW64_ENABLED);
« no previous file with comments | « chrome/browser/browser_process_impl.cc ('k') | chrome/browser/component_updater/pnacl/pnacl_component_installer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698