diff options
author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-07 22:15:34 +0000 |
---|---|---|
committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-07 22:15:34 +0000 |
commit | f4812211951630ac47b96a1b7b78f3a3dcc2b9b9 (patch) | |
tree | 68db135c222a25e8a158bdff620624aa1d75ec34 /base/sys_info_chromeos.cc | |
parent | 42e5c865f8702de0cd9e24726fe9bf69cb5cae9c (diff) | |
download | chromium_src-f4812211951630ac47b96a1b7b78f3a3dcc2b9b9.zip chromium_src-f4812211951630ac47b96a1b7b78f3a3dcc2b9b9.tar.gz chromium_src-f4812211951630ac47b96a1b7b78f3a3dcc2b9b9.tar.bz2 |
Make the windows_version.h functions threadsafe by using a singleton. Add accessors to the singleton for more values that various code wants, then convert almost everyone using OSVERSIONINFO or SYSTEM_INFO structs to calling these accessors. Declare an AtExitManager in the out-of-process test runner since it didn't have one and that breaks singleton-using code in the test executable (as opposed to in chrome.dll).
A few other minor cleanups along the way (binding of "*", shorter code, etc.). Because I ran into problems with it while modifying gcapi.cc, I cleaned up our usage of strsafe.h a bit, so that files that don't need it don't include it and files that do use STRSAFE_NO_DEPRECATE instead of a modified #include order.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6816027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80851 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/sys_info_chromeos.cc')
-rw-r--r-- | base/sys_info_chromeos.cc | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/base/sys_info_chromeos.cc b/base/sys_info_chromeos.cc index b15d71b..cd17c18 100644 --- a/base/sys_info_chromeos.cc +++ b/base/sys_info_chromeos.cc @@ -43,9 +43,9 @@ static base::LazyInstance<ChromeOSVersionNumbers> g_chrome_os_version_numbers(base::LINKER_INITIALIZED); // static -void SysInfo::OperatingSystemVersionNumbers(int32 *major_version, - int32 *minor_version, - int32 *bugfix_version) { +void SysInfo::OperatingSystemVersionNumbers(int32* major_version, + int32* minor_version, + int32* bugfix_version) { if (!g_chrome_os_version_numbers.Get().parsed) { // The other implementations of SysInfo don't block on the disk. // See http://code.google.com/p/chromium/issues/detail?id=60394 @@ -75,9 +75,9 @@ std::string SysInfo::GetLinuxStandardBaseVersionKey() { // static void SysInfo::ParseLsbRelease(const std::string& lsb_release, - int32 *major_version, - int32 *minor_version, - int32 *bugfix_version) { + int32* major_version, + int32* minor_version, + int32* bugfix_version) { size_t version_key_index = std::string::npos; for (int i = 0; kLinuxStandardBaseVersionKeys[i] != NULL; ++i) { version_key_index = lsb_release.find(kLinuxStandardBaseVersionKeys[i]); |