Fix flaky tests

Fix flaky tests

BUG=webrtc:5190, webrtc:5136
Committed: https://crrev.com/a4845efe14f14c30d2697265af68c9c1b8893ac6
Cr-Commit-Position: refs/heads/master@{#10639}

[honghaiz3, 2015-11-10 21:24:12]

Description was changed from

==========
Fix flaky tests

BUG=
==========

to

==========
Fix flaky tests

BUG=webrtc:5190
==========

[honghaiz3, 2015-11-10 21:25:58]

honghaiz@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[honghaiz3, 2015-11-10 21:30:19]

Description was changed from

==========
Fix flaky tests

BUG=webrtc:5190
==========

to

==========
Fix flaky tests

BUG=webrtc:5190,webrtc:5136
==========

[pthatcher, 2015-11-12 19:35:20]

pthatcher@google.com changed reviewers:
+ pthatcher@google.com

[pthatcher, 2015-11-12 19:35:20]

lgtm

https://codereview.webrtc.org/1405103006/diff/20001/webrtc/p2p/base/p2ptransp...
File webrtc/p2p/base/p2ptransportchannel_unittest.cc (right):

https://codereview.webrtc.org/1405103006/diff/20001/webrtc/p2p/base/p2ptransp...
webrtc/p2p/base/p2ptransportchannel_unittest.cc:1544: 1000, 1000);
Did you run "git cl format"?

[honghaiz3, 2015-11-12 19:38:05]

https://codereview.webrtc.org/1405103006/diff/20001/webrtc/p2p/base/p2ptransp...
File webrtc/p2p/base/p2ptransportchannel_unittest.cc (right):

https://codereview.webrtc.org/1405103006/diff/20001/webrtc/p2p/base/p2ptransp...
webrtc/p2p/base/p2ptransportchannel_unittest.cc:1544: 1000, 1000);
On 2015/11/12 19:35:20, pthatcher wrote:
> Did you run "git cl format"?

This is the result from that.

[honghaiz3, 2015-11-12 19:46:05]

The CQ bit was checked by honghaiz@webrtc.org

[honghaiz3, 2015-11-12 19:46:05]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@google.com
Link to the patchset: https://codereview.webrtc.org/1405103006/#ps40001 (title:
"Merge to head")

[commit-bot: I haz the power, 2015-11-12 19:46:12]

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

[commit-bot: I haz the power, 2015-11-12 19:51:10]

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

[commit-bot: I haz the power, 2015-11-12 19:51:11]

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

[honghaiz3, 2015-11-12 21:21:04]

This needs a lgtm from your webrtc account.

[pthatcher1, 2015-11-13 01:46:36]

lgtm

[honghaiz3, 2015-11-13 17:45:47]

The CQ bit was checked by honghaiz@webrtc.org

[commit-bot: I haz the power, 2015-11-13 17:45:49]

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

[commit-bot: I haz the power, 2015-11-13 17:52:59]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-13 17:53:09]

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

[tommi, 2015-11-16 12:27:28]

On 2015/11/13 at 17:53:09, commit-bot wrote:
> Patchset 3 (id:??) landed as
https://crrev.com/a4845efe14f14c30d2697265af68c9c1b8893ac6
> Cr-Commit-Position: refs/heads/master@{#10639}

drive by nit: It would be good if the description is more specific.  List what
tests, test suites or area that the tests cover.  That way the commit log is a
lot easier to parse and search.

[honghaiz3, 2015-11-16 20:48:59]

On 2015/11/16 12:27:28, tommi (webrtc) wrote:
> On 2015/11/13 at 17:53:09, commit-bot wrote:
> > Patchset 3 (id:??) landed as
> https://crrev.com/a4845efe14f14c30d2697265af68c9c1b8893ac6
> > Cr-Commit-Position: refs/heads/master@{#10639}
> 
> drive by nit: It would be good if the description is more specific.  List what
> tests, test suites or area that the tests cover.  That way the commit log is a
> lot easier to parse and search.
Will make it more specific in the future.