LoggingLock::Init() should append a non-empty string to

chrome/common/logging_chrome.cc

Index: chrome/common/logging_chrome.cc
===================================================================
--- chrome/common/logging_chrome.cc	(revision 155649)
+++ chrome/common/logging_chrome.cc	(working copy)
@@ -262,9 +262,10 @@
   IPC::Logging::set_log_function_map(&g_log_function_mapping);
 #endif
   LoggingDestination logging_dest = DetermineLogMode(command_line);
+  LogLockingState log_locking_state = LOCK_LOG_FILE;
   FilePath log_path;
 #if defined(OS_CHROMEOS)
-    FilePath target_path;
+  FilePath target_path;
 #endif
 
   // Don't resolve the log path unless we need to. Otherwise we leave an open
@@ -291,6 +292,8 @@
     // since that will remove the newly created link instead.
     delete_old_log_file = logging::APPEND_TO_OLD_LOG_FILE;
 #endif
+  } else {
+    log_locking_state = DONT_LOCK_LOG_FILE;

[wtc, 2012/09/12 20:27:09]

LogLockingState has four possible values:

enum LoggingDestination { LOG_NONE,
                          LOG_ONLY_TO_FILE,
                          LOG_ONLY_TO_SYSTEM_DEBUG_LOG,
                          LOG_TO_BOTH_FILE_AND_SYSTEM_DEBUG_LOG };

So if we get here, logging_dest must be either LOG_NONE or
LOG_ONLY_TO_SYSTEM_DEBUG_LOG.

   }
 
   logging::DcheckState dcheck_state =
@@ -300,7 +303,7 @@
 
   bool success = InitLogging(log_path.value().c_str(),
                              logging_dest,
-                             logging::LOCK_LOG_FILE,
+                             log_locking_state,
                              delete_old_log_file,
                              dcheck_state);