Cleanups in Chromoting Host.

remoting/host/usage_stats_consent_win.cc

Index: remoting/host/usage_stats_consent_win.cc
diff --git a/remoting/host/usage_stats_consent_win.cc b/remoting/host/usage_stats_consent_win.cc
index f0d0a1dc5f9a63a26b521ded2e7ffd3362eb4201..78673686e0a734652ef7dcce5a4ea243cc17f500 100644
--- a/remoting/host/usage_stats_consent_win.cc
+++ b/remoting/host/usage_stats_consent_win.cc
@@ -10,7 +10,6 @@
 #include "base/logging.h"
 #include "base/stringprintf.h"
 #include "base/win/registry.h"
-#include "remoting/host/constants.h"
 
 namespace {
 
@@ -23,11 +22,16 @@ const wchar_t kOmahaClientState[] = L"ClientState";
 const wchar_t kOmahaClientStateMedium[] = L"ClientStateMedium";
 const wchar_t kOmahaUsagestatsValue[] = L"usagestats";
 
+// The Omaha Appid of the host. It should be kept in sync with $(var.OmahaAppid)
+// defined in remoting/host/installer/chromoting.wxs and the Omaha server
+// configuration.
+const wchar_t kHostOmahaAppid[] = L"{b210701e-ffc4-49e3-932b-370728c72662}";

[alexeypa (please no reviews), 2012/09/14 15:45:26]

nit: Strictly speaking it does not belong here. branding.cc is a better, though
far from perfect, place.

[Sergey Ulanov, 2012/09/14 17:35:02]

Yes, it was a mistake to put it here. I added new remoting/host/win/omaha.h
header and put it there. It's also used in the plugin, so it has to be in a
shared header.

+
 LONG ReadUsageStatsValue(const wchar_t* state_key, DWORD* usagestats_out) {
   // presubmit: allow wstring
   std::wstring client_state = StringPrintf(kOmahaClientStateKeyFormat,
                                            state_key,
-                                           remoting::kHostOmahaAppid);
+                                           kHostOmahaAppid);
   base::win::RegKey key;
   LONG result = key.Open(HKEY_LOCAL_MACHINE, client_state.c_str(), KEY_READ);
   if (result != ERROR_SUCCESS) {