Expose WebCore log channels on the chrome command line

chrome/common/chrome_switches.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/common/chrome_switches.h"
 
 #include "base/base_switches.h"
 #include "base/command_line.h"
 
 namespace switches {
@@ skipping 1215 matching lines @@
 const char kUtilityProcessAllowedDir[]      = "utility-allowed-dir";
 
 // Print version information and quit.
 const char kVersion[]                       = "version";
 
 // Will add kWaitForDebugger to every child processes. If a value is passed, it
 // will be used as a filter to determine if the child process should have the
 // kWaitForDebugger flag passed on or not.
 const char kWaitForDebuggerChildren[]       = "wait-for-debugger-children";
 
+// Choose which logging channels in WebCore to activate.  See
+// third_party/WebKit/Source/WebCore/platform/Logging.cpp for a list of

[darin (slow to review), 2011/02/15 20:54:56]

this comment runs the risk of bit-rotting.  if someone moves that file in
WebCore, they surely won't think of fixing this comment!  that said, this
comment is definitely useful as one would like to know where to look for the
various log channel definitions.

[gavinp, 2011/02/15 21:25:48]

I made the comment less specific, so at least it'll rot more gracefully. 
Without something like this, it's hard to know what your options are, although I
anticipate most users will already have seen what they want in the WebKit
source.

+// available channels.
+const char kWebCoreLogChannels[]            = "webcore-log-channels";
+
 // Causes the worker process allocation to use as many processes as cores.
 const char kWebWorkerProcessPerCore[]       = "web-worker-process-per-core";
 
 // Causes workers to run together in one process, depending on their domains.
 // Note this is duplicated in webworkerclient_impl.cc
 const char kWebWorkerShareProcesses[]       = "web-worker-share-processes";
 
 // Use WinHTTP to fetch and evaluate PAC scripts. Otherwise the default is
 // to use Chromium's network stack to fetch, and V8 to evaluate.
 const char kWinHttpProxyResolver[]          = "winhttp-proxy-resolver";
@@ skipping 170 matching lines @@
 
 // -----------------------------------------------------------------------------
 // DO NOT ADD YOUR CRAP TO THE BOTTOM OF THIS FILE.
 //
 // You were going to just dump your switches here, weren't you? Instead,
 // please put them in alphabetical order above, or in order inside the
 // appropriate ifdef at the bottom. The order should match the header.
 // -----------------------------------------------------------------------------
 
 }  // namespace switches