Ensure syncer::internal::GetComputerName returns a UTF-8 string.

sync/util/get_session_name_win.cc

 // Copyright (c) 2012 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 "sync/util/get_session_name_win.h"
 
+#include "base/logging.h"

[Nicolas Zea, 2014/08/28 18:58:29]

nit: can remove this?

[maniscalco, 2014/08/28 19:10:58]

We need it since it's where DCHECK comes from.

+#include "base/macros.h"
+#include "base/strings/utf_string_conversions.h"
+
 #include <windows.h>
 
 namespace syncer {
 namespace internal {
 
 std::string GetComputerName() {
-  char computer_name[MAX_COMPUTERNAME_LENGTH + 1];
-  DWORD size = sizeof(computer_name);
-  if (GetComputerNameA(computer_name, &size))
-    return computer_name;
+  wchar_t computer_name[MAX_COMPUTERNAME_LENGTH + 1] = {0};
+  DWORD size = arraysize(computer_name);
+  if (::GetComputerNameW(computer_name, &size)) {
+    std::string result;
+    bool conversion_successful = base::WideToUTF8(computer_name, size, &result);
+    DCHECK(conversion_successful);
+    return result;
+  }
   return std::string();
 }
 
 }  // namespace internal
 }  // namespace syncer