Delay based logging.

Delay based logging.

BUG=none

Review-Url: https://codereview.webrtc.org/2808833002
Cr-Commit-Position: refs/heads/master@{#17641}
Committed: https://chromium.googlesource.com/external/webrtc/+/10fc0e6385b812981f6b64f611259887ca14de51

[philipel, 2017-04-10 13:20:44]

philipel@webrtc.org changed reviewers:
+ terelius@webrtc.org

[philipel, 2017-04-10 13:20:45]

https://codereview.webrtc.org/2808833002/diff/1/webrtc/tools/event_log_visual...
File webrtc/tools/event_log_visualizer/plot_python.cc (right):

https://codereview.webrtc.org/2808833002/diff/1/webrtc/tools/event_log_visual...
webrtc/tools/event_log_visualizer/plot_python.cc:81: "marker='o', ls=' ')\n",
The '.' format is really hard to see, 'o' is much better :)

[philipel, 2017-04-10 13:24:21]

The CQ bit was checked by philipel@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-10 13:24:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-04-10 13:36:55]

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

[commit-bot: I haz the power, 2017-04-10 13:36:56]

Dry run: Try jobs failed on following builders:
  linux_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_rel/builds/25100)

[terelius, 2017-04-10 13:46:27]

lgtm

https://codereview.webrtc.org/2808833002/diff/1/webrtc/tools/event_log_visual...
File webrtc/tools/event_log_visualizer/plot_python.cc (right):

https://codereview.webrtc.org/2808833002/diff/1/webrtc/tools/event_log_visual...
webrtc/tools/event_log_visualizer/plot_python.cc:81: "marker='o', ls=' ')\n",
On 2017/04/10 13:20:44, philipel wrote:
> The '.' format is really hard to see, 'o' is much better :)

Is this also true for graphs in which every point is highlighted?

[philipel, 2017-04-10 13:55:09]

https://codereview.webrtc.org/2808833002/diff/1/webrtc/tools/event_log_visual...
File webrtc/tools/event_log_visualizer/plot_python.cc (right):

https://codereview.webrtc.org/2808833002/diff/1/webrtc/tools/event_log_visual...
webrtc/tools/event_log_visualizer/plot_python.cc:81: "marker='o', ls=' ')\n",
On 2017/04/10 13:46:27, terelius wrote:
> On 2017/04/10 13:20:44, philipel wrote:
> > The '.' format is really hard to see, 'o' is much better :)
> 
> Is this also true for graphs in which every point is highlighted? 

So far DOT_GRAPH is only used to visualize probing events, so everything else
should look normal.

[philipel, 2017-04-10 15:12:52]

The CQ bit was checked by philipel@webrtc.org

[commit-bot: I haz the power, 2017-04-10 15:13:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-04-10 15:21:25]

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

[commit-bot: I haz the power, 2017-04-10 15:21:26]

Try jobs failed on following builders:
  mac_asan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_asan/builds/23787)

[philipel, 2017-04-10 15:36:33]

The CQ bit was checked by philipel@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-10 15:36:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-04-10 17:37:08]

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

[commit-bot: I haz the power, 2017-04-10 17:37:09]

Dry run: Try jobs failed on following builders:
  win_baremetal on master.tryserver.webrtc (JOB_TIMED_OUT, build hasn't started
yet, builder probably lacks capacity)

[philipel, 2017-04-11 08:47:49]

The CQ bit was checked by philipel@webrtc.org

[philipel, 2017-04-11 08:47:50]

The patchset sent to the CQ was uploaded after l-g-t-m from terelius@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2808833002/#ps20001 (title:
"Unittest fix.")

[commit-bot: I haz the power, 2017-04-11 08:47:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-04-11 08:50:24]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491900469916380,
"parent_rev": "64e739aeae5629cbbebf2a19e1d3e6b452bb6d0b", "commit_rev":
"10fc0e6385b812981f6b64f611259887ca14de51"}

[commit-bot: I haz the power, 2017-04-11 08:50:27]

Description was changed from

==========
Delay based logging.

BUG=none
==========

to

==========
Delay based logging.

BUG=none

Review-Url: https://codereview.webrtc.org/2808833002
Cr-Commit-Position: refs/heads/master@{#17641}
Committed:
https://chromium.googlesource.com/external/webrtc/+/10fc0e6385b812981f6b64f61...
==========

[commit-bot: I haz the power, 2017-04-11 08:50:28]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/10fc0e6385b812981f6b64f61...