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

Side by Side Diff: chrome/browser/component_updater/component_updater_utils.cc

Issue 271573003: The component update checks should include the amount of installed memory. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/component_updater/component_updater_utils.h" 5 #include "chrome/browser/component_updater/component_updater_utils.h"
6 6
7 #include <cmath>
8
7 #include "base/file_util.h" 9 #include "base/file_util.h"
8 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
9 #include "base/guid.h" 11 #include "base/guid.h"
10 #include "base/strings/string_number_conversions.h" 12 #include "base/strings/string_number_conversions.h"
11 #include "base/strings/string_piece.h" 13 #include "base/strings/string_piece.h"
12 #include "base/strings/string_util.h" 14 #include "base/strings/string_util.h"
13 #include "base/strings/stringprintf.h" 15 #include "base/strings/stringprintf.h"
14 #include "base/sys_info.h" 16 #include "base/sys_info.h"
15 #include "base/win/windows_version.h" 17 #include "base/win/windows_version.h"
16 #include "chrome/browser/component_updater/component_updater_service.h" 18 #include "chrome/browser/component_updater/component_updater_service.h"
17 #include "chrome/browser/component_updater/crx_update_item.h" 19 #include "chrome/browser/component_updater/crx_update_item.h"
18 #include "chrome/browser/omaha_query_params/omaha_query_params.h" 20 #include "chrome/browser/omaha_query_params/omaha_query_params.h"
19 #include "chrome/common/chrome_version_info.h" 21 #include "chrome/common/chrome_version_info.h"
20 #include "extensions/common/extension.h" 22 #include "extensions/common/extension.h"
21 #include "net/base/load_flags.h" 23 #include "net/base/load_flags.h"
22 #include "net/url_request/url_fetcher.h" 24 #include "net/url_request/url_fetcher.h"
23 #include "net/url_request/url_request_context_getter.h" 25 #include "net/url_request/url_request_context_getter.h"
24 #include "net/url_request/url_request_status.h" 26 #include "net/url_request/url_request_status.h"
25 27
26 namespace component_updater { 28 namespace component_updater {
27 29
30 namespace {
31
32 // Returns the amount of physical memory in GB, rounded to the nearest GB.
33 int GetPhysicalMemoryGB() {
34 const double kOneGB = 1024 * 1024 * 1024;
35 const int64 phys_mem = base::SysInfo::AmountOfPhysicalMemory();
36 return static_cast<int>(std::floor(0.5 + phys_mem / kOneGB));
37 }
38
39 } // namespace
40
28 std::string BuildProtocolRequest(const std::string& request_body, 41 std::string BuildProtocolRequest(const std::string& request_body,
29 const std::string& additional_attributes) { 42 const std::string& additional_attributes) {
30 const std::string prod_id(chrome::OmahaQueryParams::GetProdIdString( 43 const std::string prod_id(chrome::OmahaQueryParams::GetProdIdString(
31 chrome::OmahaQueryParams::CHROME)); 44 chrome::OmahaQueryParams::CHROME));
32 const chrome::VersionInfo chrome_version_info; 45 const chrome::VersionInfo chrome_version_info;
33 const std::string chrome_version(chrome_version_info.Version()); 46 const std::string chrome_version(chrome_version_info.Version());
34 47
35 std::string request( 48 std::string request(
36 "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" 49 "<?xml version=\"1.0\" encoding=\"UTF-8\"?>"
37 "<request protocol=\"3.0\" "); 50 "<request protocol=\"3.0\" ");
(...skipping 18 matching lines...) Expand all
56 chrome::OmahaQueryParams::GetArch(), // "arch" 69 chrome::OmahaQueryParams::GetArch(), // "arch"
57 chrome::OmahaQueryParams::GetNaclArch()); // "nacl_arch" 70 chrome::OmahaQueryParams::GetNaclArch()); // "nacl_arch"
58 #if defined(OS_WIN) 71 #if defined(OS_WIN)
59 const bool is_wow64(base::win::OSInfo::GetInstance()->wow64_status() == 72 const bool is_wow64(base::win::OSInfo::GetInstance()->wow64_status() ==
60 base::win::OSInfo::WOW64_ENABLED); 73 base::win::OSInfo::WOW64_ENABLED);
61 if (is_wow64) 74 if (is_wow64)
62 base::StringAppendF(&request, " wow64=\"1\""); 75 base::StringAppendF(&request, " wow64=\"1\"");
63 #endif 76 #endif
64 base::StringAppendF(&request, ">"); 77 base::StringAppendF(&request, ">");
65 78
79 // HW platform information.
80 base::StringAppendF(&request,
81 "<hw physmemory=\"%d\"/>",
82 GetPhysicalMemoryGB()); // "physmem" in GB.
83
66 // OS version and platform information. 84 // OS version and platform information.
67 base::StringAppendF( 85 base::StringAppendF(
68 &request, 86 &request,
69 "<os platform=\"%s\" version=\"%s\" arch=\"%s\"/>", 87 "<os platform=\"%s\" version=\"%s\" arch=\"%s\"/>",
70 chrome::VersionInfo().OSType().c_str(), // "platform" 88 chrome::VersionInfo().OSType().c_str(), // "platform"
71 base::SysInfo().OperatingSystemVersion().c_str(), // "version" 89 base::SysInfo().OperatingSystemVersion().c_str(), // "version"
72 base::SysInfo().OperatingSystemArchitecture().c_str()); // "arch" 90 base::SysInfo().OperatingSystemArchitecture().c_str()); // "arch"
73 91
74 // The actual payload of the request. 92 // The actual payload of the request.
75 base::StringAppendF(&request, "%s</request>", request_body.c_str()); 93 base::StringAppendF(&request, "%s</request>", request_body.c_str());
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
163 DCHECK(extensions::Extension::IdIsValid(id)); 181 DCHECK(extensions::Extension::IdIsValid(id));
164 return id; 182 return id;
165 } 183 }
166 184
167 std::string GetCrxComponentID(const CrxComponent& component) { 185 std::string GetCrxComponentID(const CrxComponent& component) {
168 return HexStringToID(StringToLowerASCII( 186 return HexStringToID(StringToLowerASCII(
169 base::HexEncode(&component.pk_hash[0], component.pk_hash.size() / 2))); 187 base::HexEncode(&component.pk_hash[0], component.pk_hash.size() / 2)));
170 } 188 }
171 189
172 } // namespace component_updater 190 } // namespace component_updater
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698