[Blimp Client] Enables verbose log for incoming and outgoing messages by default

blimp/client/app/blimp_startup.cc

Index: blimp/client/app/blimp_startup.cc
diff --git a/blimp/client/app/blimp_startup.cc b/blimp/client/app/blimp_startup.cc
index e19ee3f0617976248820b59850f40033cf5e1637..0c2b6ecf833f13fdbfa07a10e64f88bd427f3e6b 100644
--- a/blimp/client/app/blimp_startup.cc
+++ b/blimp/client/app/blimp_startup.cc
@@ -4,6 +4,7 @@
 
 #include "blimp/client/app/blimp_startup.h"
 
+#include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ -33,6 +34,13 @@ namespace blimp {
 namespace client {
 
 void InitializeLogging() {
+  // TODO(haibinlu): Remove this before release.
+  // Enables a few verbose log by default.
+  if (!base::CommandLine::ForCurrentProcess()->HasSwitch("vmodule")) {
+    base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
+        "vmodule", "blimp_message_pump=1, blimp_connection=1");

[Khushal, 2016/03/29 21:10:04]

Should this be *blimp*? I think we have bugs for adding logs for multiple
features as well.

[Kevin M, 2016/03/29 21:11:45]

I don't think blimp* is a good choice because we have a lot of other features
whose module names aren't prefixed by "blimp", so we'd end up having to do a lot
of whitelisting anyways.

[haibinlu1, 2016/03/29 21:11:47]

there will be way too many logs. feature shall change DVLOG to VLOG and
explicitly add itself there.

+  }
+
   logging::LoggingSettings settings;
 #if defined(OS_ANDROID)
   settings.logging_dest = logging::LOG_TO_SYSTEM_DEBUG_LOG;