Stop writing logs to %ProgramData%\Chromoting\debug.log (Windows only).

remoting/host/logging_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 "remoting/host/logging.h"
 
 #include <guiddef.h>
 
-#include "base/file_path.h"
 #include "base/logging.h"
 #include "base/logging_win.h"
-#include "remoting/host/branding.h"
 
 // {2db51ca1-4fd8-4b88-b5a2-fb8606b66b02}
 const GUID kRemotingHostLogProvider =
     { 0x2db51ca1, 0x4fd8, 0x4b88,
         { 0xb5, 0xa2, 0xfb, 0x86, 0x06, 0xb6, 0x6b, 0x02 } };
 
 namespace remoting {
 
 void InitHostLogging() {
-#if defined(NDEBUG)
-  // Write logs to the system debug log in release build.
+  // Write logs to the system debug log.
   logging::InitLogging(
       NULL,
       logging::LOG_ONLY_TO_SYSTEM_DEBUG_LOG,
-      logging::LOCK_LOG_FILE,
+      logging::DONT_LOCK_LOG_FILE,
       logging::DELETE_OLD_LOG_FILE,
       logging::DISABLE_DCHECK_FOR_NON_OFFICIAL_RELEASE_BUILDS);
-#else  // !defined(NDEBUG)
-  // Write logs to a file in debug build.
-  FilePath debug_log = remoting::GetConfigDir().
-      Append(FILE_PATH_LITERAL("debug.log"));
-  logging::InitLogging(
-      debug_log.value().c_str(),
-      logging::LOG_ONLY_TO_FILE,
-      logging::LOCK_LOG_FILE,
-      logging::DELETE_OLD_LOG_FILE,
-      logging::DISABLE_DCHECK_FOR_NON_OFFICIAL_RELEASE_BUILDS);
-#endif  // !defined(NDEBUG)

[Jamie, 2013/01/07 18:31:21]

This disables logging entirely for debug builds, correct? Couldn't we instead
log with best-effort to, for example, C:\TEMP? That would save each developer
adding logging to their local checkouts to help with debugging--we'd just need
to create the directory instead.

[alexeypa (please no reviews), 2013/01/07 19:10:31]

This disables logging to a file. Logging to ETW isn't going anywhere. 
I don't think it is a good idea. First, all sorts of temp folders are not
accessible to low integrity processes as well. At least their parent folders are
not accessible to low integrity processes. There is a special AppData\LocalLow
but it is not going to be any more convenient to use than ETW.

 
   // Enable trace control and transport through event tracing for Windows.
   logging::LogEventProvider::Initialize(kRemotingHostLogProvider);
 }
 
 }  // namespace remoting