Remove trace event overhead reporting feature

Remove trace event overhead reporting feature

It's hard to make it right and useful. It may be also a source of bugs.
Better to keep things simple.

Committed: https://crrev.com/b49d6eda590f046695a461334feda2da72c4fc8b
Cr-Commit-Position: refs/heads/master@{#349491}

[Xianzhu, 2015-09-16 21:45:01]

wangxianzhu@chromium.org changed reviewers:
+ dsinclair@chromium.org, nduca@chromium.org

[dsinclair, 2015-09-17 13:12:28]

https://codereview.chromium.org/1345223004/diff/1/base/trace_event/trace_log.cc
File base/trace_event/trace_log.cc (right):

https://codereview.chromium.org/1345223004/diff/1/base/trace_event/trace_log....
base/trace_event/trace_log.cc:247: int event_count_;
Can we also remove event_count_?

[Xianzhu, 2015-09-17 16:01:49]

https://codereview.chromium.org/1345223004/diff/1/base/trace_event/trace_log.cc
File base/trace_event/trace_log.cc (right):

https://codereview.chromium.org/1345223004/diff/1/base/trace_event/trace_log....
base/trace_event/trace_log.cc:247: int event_count_;
On 2015/09/17 13:12:28, dsinclair wrote:
> Can we also remove event_count_?

Done.

[dsinclair, 2015-09-17 16:21:31]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2015-09-17 16:21:32]

lgtm

[commit-bot: I haz the power, 2015-09-17 16:40:19]

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

[commit-bot: I haz the power, 2015-09-17 17:27:57]

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

[commit-bot: I haz the power, 2015-09-17 17:27:58]

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

[Xianzhu, 2015-09-17 19:40:44]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2015-09-17 19:41:51]

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

[Xianzhu, 2015-09-17 20:42:27]

Patchset #3 (id:40001) has been deleted

[Xianzhu, 2015-09-17 20:42:54]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2015-09-17 20:43:59]

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

[commit-bot: I haz the power, 2015-09-17 20:50:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-17 20:50:45]

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