Make the windows_version.h functions threadsafe by using a singleton.

chrome/browser/bug_report_util.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/bug_report_util.h"
 
 #include <sstream>
 #include <string>
 
 #include "base/command_line.h"
 #include "base/file_version_info.h"
 #include "base/file_util.h"
 #include "base/memory/singleton.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
+#include "base/win/windows_version.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/browser_process_impl.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/safe_browsing/safe_browsing_util.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/net/url_fetcher.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "googleurl/src/gurl.h"
 #include "grit/generated_resources.h"
@@ skipping 123 matching lines @@
   LOG(WARNING) << "FEEDBACK: Submission to feedback server (" << url <<
       ") status: " << error_stream.str() << std::endl;
 
   // Delete the URLFetcher.
   delete source;
   // And then delete ourselves.
   delete this;
 }
 
 // static
-void BugReportUtil::SetOSVersion(std::string *os_version) {
+void BugReportUtil::SetOSVersion(std::string* os_version) {
 #if defined(OS_WIN)
-  OSVERSIONINFO osvi;
-  ZeroMemory(&osvi, sizeof(OSVERSIONINFO));
-  osvi.dwOSVersionInfoSize = sizeof(OSVERSIONINFO);
-
-  if (GetVersionEx(&osvi)) {
-    *os_version = StringPrintf("%d.%d.%d %S",
-      osvi.dwMajorVersion,
-      osvi.dwMinorVersion,
-      osvi.dwBuildNumber,
-      osvi.szCSDVersion);
-  } else {
-    *os_version = "unknown";
-  }
+  OSInfo* os_info = base::win::OSInfo::GetInstance();
+  const int* version = os_info->version_number();
+  *os_version = StringPrintf("%d.%d.%d", version[0], version[1], version[2]);
+  int service_pack = os_info->service_pack()[0];
+  if (service_pack > 0)
+    os_version->append(StringPrintf("Service Pack %d", service_pack));
 #elif defined(OS_MACOSX)
   int32 major;
   int32 minor;
   int32 bugFix;
   base::SysInfo::OperatingSystemVersionNumbers(&major, &minor, &bugFix);
   *os_version = StringPrintf("%d.%d.%d", major, minor, bugFix);
 #else
   *os_version = "unknown";
 #endif
 }
@@ skipping 200 matching lines @@
 
 // static
 void BugReportUtil::ReportPhishing(TabContents* currentTab,
                                    const std::string& phishing_url) {
   currentTab->controller().LoadURL(
       safe_browsing_util::GeneratePhishingReportUrl(
           kReportPhishingUrl, phishing_url),
       GURL(),
       PageTransition::LINK);
 }