fix bug 5430

fix bug 5430

Fixed misusage of Connection function and also fixed the test case.

BUG=webrtc:5430
Committed: https://crrev.com/56271ed8894fb2e985c9b366e4e38313a4652d63
Cr-Commit-Position: refs/heads/master@{#11278}

[guoweis_webrtc, 2016-01-15 18:11:07]

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

[guoweis_webrtc, 2016-01-15 18:11:08]

PTAL.

[guoweis_webrtc, 2016-01-15 18:11:53]

Description was changed from

==========
fix bug 5430

BUG=webrtc:5430
==========

to

==========
fix bug 5430

Fixed misusage of Connection function and also fixed the test case.

BUG=webrtc:5430
==========

[pthatcher1, 2016-01-15 19:30:52]

lgtm

https://codereview.webrtc.org/1592763003/diff/1/webrtc/base/network_unittest.cc
File webrtc/base/network_unittest.cc (right):

https://codereview.webrtc.org/1592763003/diff/1/webrtc/base/network_unittest....
webrtc/base/network_unittest.cc:954: std::vector<Network*> networks;
Should we EXPECT_TRUE(!networks.empty()) ?

[guoweis_webrtc, 2016-01-15 21:41:28]

Patchset #3 (id:40001) has been deleted

[guoweis_webrtc, 2016-01-15 21:56:55]

The CQ bit was checked by guoweis@webrtc.org

[guoweis_webrtc, 2016-01-15 21:56:55]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1592763003/#ps60001 (title:
"Fix ipv6 test failure.")

[commit-bot: I haz the power, 2016-01-15 21:57:05]

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

[commit-bot: I haz the power, 2016-01-15 22:45:08]

Description was changed from

==========
fix bug 5430

Fixed misusage of Connection function and also fixed the test case.

BUG=webrtc:5430
==========

to

==========
fix bug 5430

Fixed misusage of Connection function and also fixed the test case.

BUG=webrtc:5430
==========

[commit-bot: I haz the power, 2016-01-15 22:45:09]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-01-15 22:45:17]

Description was changed from

==========
fix bug 5430

Fixed misusage of Connection function and also fixed the test case.

BUG=webrtc:5430
==========

to

==========
fix bug 5430

Fixed misusage of Connection function and also fixed the test case.

BUG=webrtc:5430

Committed: https://crrev.com/56271ed8894fb2e985c9b366e4e38313a4652d63
Cr-Commit-Position: refs/heads/master@{#11278}
==========

[commit-bot: I haz the power, 2016-01-15 22:45:18]

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