Oilpan: Add TRACE_DISABLED_BY_DEFAULT to trace counters in oilpan

Oilpan: Add TRACE_DISABLED_BY_DEFAULT to trace counters in oilpan

Those trace counters are messing up the top of the UI of about:tracing.
Since the counters are interesting to only oilpan developers,
this CL disables the counters (except for four important counters).

See https://github.com/catapult-project/catapult/issues/1528 and https://groups.google.com/a/chromium.org/forum/#!topic/oilpan-reviews/AGegKLIIou8 for more details.

Committed: https://crrev.com/9da9d03f47fb41dcf88931dfed8dac08d90778b2
Cr-Commit-Position: refs/heads/master@{#354184}

[haraken, 2015-09-29 12:25:52]

haraken@chromium.org changed reviewers:
+ keishi@chromium.org, oilpan-reviews@chromium.org, petrcermak@chromium.org

[haraken, 2015-09-29 12:25:53]

PTAL

[petrcermak, 2015-09-29 12:34:48]

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

[petrcermak, 2015-09-29 12:34:48]

LGTM.

nduca: Could you please double-check that this approach is in line with the rest
of tracing?

One nit in the patch description: s/counters is/counters are/

I think it would be a good idea to add links to the Trace Viewer bug
(https://github.com/catapult-project/catapult/issues/1528) and our discussion
(https://groups.google.com/a/chromium.org/forum/#!topic/oilpan-reviews/AGegKLI...)
to the patch description (instead of an empty "BUG=" line).

Thanks,
Petr

[nduca, 2015-09-29 12:36:18]

lgtm thanks haraken!

[haraken, 2015-09-29 12:37:47]

> One nit in the patch description: s/counters is/counters are/
> 
> I think it would be a good idea to add links to the Trace Viewer bug
> (https://github.com/catapult-project/catapult/issues/1528) and our discussion
>
(https://groups.google.com/a/chromium.org/forum/#!topic/oilpan-reviews/AGegKLI...)
> to the patch description (instead of an empty "BUG=" line).
> 
> Thanks,
> Petr

Done.

[haraken, 2015-09-29 12:38:24]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2015-09-29 12:40:09]

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

[sof, 2015-09-29 12:43:45]

sigbjornf@opera.com changed reviewers:
+ sigbjornf@opera.com

[sof, 2015-09-29 12:43:45]

lgtm

[commit-bot: I haz the power, 2015-09-29 14:02:08]

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

[commit-bot: I haz the power, 2015-09-29 14:02:09]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[haraken, 2015-10-06 06:16:01]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2015-10-06 06:16:12]

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

[commit-bot: I haz the power, 2015-10-06 06:17:29]

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

[commit-bot: I haz the power, 2015-10-06 06:17:30]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[haraken, 2015-10-14 22:52:56]

The CQ bit was checked by haraken@chromium.org

[haraken, 2015-10-14 22:52:57]

The patchset sent to the CQ was uploaded after l-g-t-m from sigbjornf@opera.com,
nduca@chromium.org, petrcermak@chromium.org
Link to the patchset: https://codereview.chromium.org/1372313003/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-10-14 22:53:59]

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

[commit-bot: I haz the power, 2015-10-15 01:08:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-15 01:10:09]

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