Use disabled-by-default prefix in logging NetLog events to TraceLog

Use disabled-by-default prefix in logging NetLog events to TraceLog

BUG=399701
Committed: https://crrev.com/1d2e6f597e27f549dd27227396d0838b9228d85f
Cr-Commit-Position: refs/heads/master@{#300543}

[xunjieli, 2014-10-21 17:40:19]

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

[xunjieli, 2014-10-21 17:40:20]

Hi Matt,I forgot we should prefix "netlog" category with a "disabled-by-default"
prefix. As nduca@ mentioned in the email thread, we probably should not enable
it by default.
PTAL. Thanks!

[nduca, 2014-10-21 17:50:07]

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

[nduca, 2014-10-21 17:50:08]

some nits, lgtm when resolved

https://codereview.chromium.org/671533005/diff/1/net/base/trace_net_log_obser...
File net/base/trace_net_log_observer_unittest.cc (right):

https://codereview.chromium.org/671533005/diff/1/net/base/trace_net_log_obser...
net/base/trace_net_log_observer_unittest.cc:31: const char
kNetLogTracingCategory[] = "disabled-by-default-netlog";
can you use the trace macro to form this? TRACE_DISABLED_BY_DEFAULT("netlog") i
think is how its written?

[xunjieli, 2014-10-21 17:55:15]

https://codereview.chromium.org/671533005/diff/1/net/base/trace_net_log_obser...
File net/base/trace_net_log_observer_unittest.cc (right):

https://codereview.chromium.org/671533005/diff/1/net/base/trace_net_log_obser...
net/base/trace_net_log_observer_unittest.cc:31: const char
kNetLogTracingCategory[] = "disabled-by-default-netlog";
On 2014/10/21 17:50:07, nduca wrote:
> can you use the trace macro to form this? TRACE_DISABLED_BY_DEFAULT("netlog")
i
> think is how its written?

Done. Thanks!

[mmenke, 2014-10-21 18:36:02]

LGTM

[xunjieli, 2014-10-21 18:36:38]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2014-10-21 18:37:55]

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

[commit-bot: I haz the power, 2014-10-21 20:05:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-21 20:06:24]

Patchset 2 (id:??) landed as
https://crrev.com/1d2e6f597e27f549dd27227396d0838b9228d85f
Cr-Commit-Position: refs/heads/master@{#300543}