Make --log-net-log and NetLogLogger hide cookies by default,

Make --log-net-log and NetLogLogger hide cookies by default,
and respect yjr --net_log_level switch.

Also remove base log level from NetLog, as it doesn't really
serve any purpose any more.

BUG=418574
Committed: https://crrev.com/42f9b97aea853b9d89dee89862af6821bc2246a1
Cr-Commit-Position: refs/heads/master@{#297530}

[mmenke, 2014-09-29 17:52:40]

Patchset #5 (id:80001) has been deleted

[mmenke, 2014-09-29 17:52:47]

Patchset #4 (id:60001) has been deleted

[mmenke, 2014-09-29 21:40:43]

mmenke@chromium.org changed reviewers:
+ davidben@chromium.org

[mmenke, 2014-09-29 21:40:44]

The new net/net_log_logger test may be overkill, but I'm paranoid.

[davidben, 2014-09-30 17:33:50]

lgtm with one comment.

https://codereview.chromium.org/612023003/diff/120001/net/base/net_log_logger...
File net/base/net_log_logger_unittest.cc (right):

https://codereview.chromium.org/612023003/diff/120001/net/base/net_log_logger...
net/base/net_log_logger_unittest.cc:17: namespace {
I've always been unclear on whether tests should be in an anonymous namespace or
not. Picking a random other one (url_request_unittest.cc), the answer seems to
be no though.

[mmenke, 2014-09-30 18:58:03]

On 2014/09/30 17:33:50, David Benjamin wrote:
> lgtm with one comment.
> 
>
https://codereview.chromium.org/612023003/diff/120001/net/base/net_log_logger...
> File net/base/net_log_logger_unittest.cc (right):
> 
>
https://codereview.chromium.org/612023003/diff/120001/net/base/net_log_logger...
> net/base/net_log_logger_unittest.cc:17: namespace {
> I've always been unclear on whether tests should be in an anonymous namespace
or
> not. Picking a random other one (url_request_unittest.cc), the answer seems to
> be no though.

My general feeling is that they should be, but I don't feel strongly about it. 
Skimming over net/ files, most aren't, but a significant number are (Maybe
around 1/4th or 1/3rd)?

https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/MwPIo2BnVhM
has a short discussion on the topic.  It's illegal for tests using TEST_F to
have the same name, but not TEST, apparently, but both probably confuse our
retry scripts, anyways.  Will Chan chimes in support of it, and I defer to him. 
:)

[mmenke, 2014-09-30 19:03:16]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2014-09-30 19:04:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/612023003/120001

[commit-bot: I haz the power, 2014-09-30 19:15:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-30 19:15:05]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[mmenke, 2014-09-30 22:19:51]

mmenke@chromium.org changed reviewers:
+ zea@chromium.org

[mmenke, 2014-09-30 22:20:08]

Zea:  Please review GCM changes

[Nicolas Zea, 2014-09-30 22:23:43]

gcm lgtm

[mmenke, 2014-09-30 22:27:36]

On 2014/09/30 22:23:43, Nicolas Zea wrote:
> gcm lgtm

Thanks!

[mmenke, 2014-09-30 22:27:40]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2014-09-30 22:29:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/612023003/120001

[commit-bot: I haz the power, 2014-09-30 22:33:53]

Committed patchset #5 (id:120001) as 26cd1c27f2c1787f482155795cf93ca827bf3373

[commit-bot: I haz the power, 2014-09-30 22:34:35]

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