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

so that users does not have to specify command line arguments.

It generates logs:

V/chromium(22041): [VERBOSE1:blimp_message_pump.cc(59)] Received <BlimpMessage type=COMPOSITOR target_tab_id=0 byte_size=61379>
V/chromium(22041): [VERBOSE1:blimp_connection.cc(66)] Sending <BlimpMessage type=PROTOCOL_CONTROL subtype=CHECKPOINT_ACK checkpoint_id=20 byte_size=15>

BUG=596796
Committed: https://crrev.com/5ae0ee4d57c0da9a27ff1f44d4346ba5c16e2a66
Cr-Commit-Position: refs/heads/master@{#383840}

[haibinlu, 2016-03-29 20:47:51]

Description was changed from

==========
[Blimp Net] Turn verbose log into DLOG(INFO) so that incoming and outgoing
messages are logged by default

This enables on both Client and Engine since net is shared.

BUG=596796
==========

to

==========
so that users does not have to specify command line arguments.

It generates logs:

V/chromium(22041): [VERBOSE1:blimp_message_pump.cc(59)] Received <BlimpMessage
type=COMPOSITOR target_tab_id=0 byte_size=61379>
V/chromium(22041): [VERBOSE1:blimp_connection.cc(66)] Sending <BlimpMessage
type=PROTOCOL_CONTROL subtype=CHECKPOINT_ACK checkpoint_id=20 byte_size=15>


BUG=596796
==========

[haibinlu, 2016-03-29 21:04:07]

haibinlu@chromium.org changed reviewers:
+ kmarshall@chromium.org

[Khushal, 2016-03-29 21:10:03]

khushalsagar@chromium.org changed reviewers:
+ khushalsagar@chromium.org

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

https://codereview.chromium.org/1841183002/diff/40001/blimp/client/app/blimp_...
File blimp/client/app/blimp_startup.cc (right):

https://codereview.chromium.org/1841183002/diff/40001/blimp/client/app/blimp_...
blimp/client/app/blimp_startup.cc:41: "vmodule", "blimp_message_pump=1,
blimp_connection=1");
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]

lgtm

https://codereview.chromium.org/1841183002/diff/40001/blimp/client/app/blimp_...
File blimp/client/app/blimp_startup.cc (right):

https://codereview.chromium.org/1841183002/diff/40001/blimp/client/app/blimp_...
blimp/client/app/blimp_startup.cc:41: "vmodule", "blimp_message_pump=1,
blimp_connection=1");
On 2016/03/29 21:10:04, Khushal wrote:
> Should this be *blimp*? I think we have bugs for adding logs for multiple
> features as well.

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:46]

haibinlu@google.com changed reviewers:
+ haibinlu@google.com

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

https://codereview.chromium.org/1841183002/diff/40001/blimp/client/app/blimp_...
File blimp/client/app/blimp_startup.cc (right):

https://codereview.chromium.org/1841183002/diff/40001/blimp/client/app/blimp_...
blimp/client/app/blimp_startup.cc:41: "vmodule", "blimp_message_pump=1,
blimp_connection=1");
On 2016/03/29 21:10:04, Khushal wrote:
> Should this be *blimp*? I think we have bugs for adding logs for multiple
> features as well.

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

[haibinlu, 2016-03-29 21:12:54]

haibinlu@chromium.org changed reviewers:
- haibinlu@google.com

[haibinlu, 2016-03-29 21:13:06]

The CQ bit was checked by haibinlu@chromium.org

[commit-bot: I haz the power, 2016-03-29 21:13:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1841183002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1841183002/40001

[commit-bot: I haz the power, 2016-03-29 22:01:18]

Description was changed from

==========
so that users does not have to specify command line arguments.

It generates logs:

V/chromium(22041): [VERBOSE1:blimp_message_pump.cc(59)] Received <BlimpMessage
type=COMPOSITOR target_tab_id=0 byte_size=61379>
V/chromium(22041): [VERBOSE1:blimp_connection.cc(66)] Sending <BlimpMessage
type=PROTOCOL_CONTROL subtype=CHECKPOINT_ACK checkpoint_id=20 byte_size=15>


BUG=596796
==========

to

==========
so that users does not have to specify command line arguments.

It generates logs:

V/chromium(22041): [VERBOSE1:blimp_message_pump.cc(59)] Received <BlimpMessage
type=COMPOSITOR target_tab_id=0 byte_size=61379>
V/chromium(22041): [VERBOSE1:blimp_connection.cc(66)] Sending <BlimpMessage
type=PROTOCOL_CONTROL subtype=CHECKPOINT_ACK checkpoint_id=20 byte_size=15>


BUG=596796
==========

[commit-bot: I haz the power, 2016-03-29 22:01:19]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-29 22:02:44]

Description was changed from

==========
so that users does not have to specify command line arguments.

It generates logs:

V/chromium(22041): [VERBOSE1:blimp_message_pump.cc(59)] Received <BlimpMessage
type=COMPOSITOR target_tab_id=0 byte_size=61379>
V/chromium(22041): [VERBOSE1:blimp_connection.cc(66)] Sending <BlimpMessage
type=PROTOCOL_CONTROL subtype=CHECKPOINT_ACK checkpoint_id=20 byte_size=15>


BUG=596796
==========

to

==========
so that users does not have to specify command line arguments.

It generates logs:

V/chromium(22041): [VERBOSE1:blimp_message_pump.cc(59)] Received <BlimpMessage
type=COMPOSITOR target_tab_id=0 byte_size=61379>
V/chromium(22041): [VERBOSE1:blimp_connection.cc(66)] Sending <BlimpMessage
type=PROTOCOL_CONTROL subtype=CHECKPOINT_ACK checkpoint_id=20 byte_size=15>

BUG=596796

Committed: https://crrev.com/5ae0ee4d57c0da9a27ff1f44d4346ba5c16e2a66
Cr-Commit-Position: refs/heads/master@{#383840}
==========

[commit-bot: I haz the power, 2016-03-29 22:02:45]

Patchset 3 (id:??) landed as
https://crrev.com/5ae0ee4d57c0da9a27ff1f44d4346ba5c16e2a66
Cr-Commit-Position: refs/heads/master@{#383840}