Do not rely on specific ordering on generated candidates in TestGetAllPortsPortRange

Do not rely on specific ordering on generated candidates in TestGetAllPortsPortRange

This fixes another WebRTC swarming tests.

BUG=webrtc:6500
Committed: https://crrev.com/a73df559d63b74acc1aeb8d9a4a8d26af6e6c6db
Cr-Commit-Position: refs/heads/master@{#14804}

[honghaiz3, 2016-10-25 21:41:39]

Description was changed from

==========
Fix more WebRTC swarming tests.

BUG=webrtc:6500
==========

to

==========
Fix another WebRTC swarming tests.

BUG=webrtc:6500
==========

[honghaiz3, 2016-10-25 21:42:25]

Patchset #1 (id:1) has been deleted

[Taylor Brandstetter, 2016-10-27 16:54:08]

deadbeef@webrtc.org changed reviewers:
+ deadbeef@webrtc.org

[Taylor Brandstetter, 2016-10-27 16:54:09]

lgtm. Though I think the description/title should be changed to "Don't rely on a
specific ordering of ports in TestGetAllPortsPortRange."

https://codereview.webrtc.org/2450983002/diff/20001/webrtc/p2p/client/basicpo...
File webrtc/p2p/client/basicportallocator_unittest.cc (right):

https://codereview.webrtc.org/2450983002/diff/20001/webrtc/p2p/client/basicpo...
webrtc/p2p/client/basicportallocator_unittest.cc:926: // Check the port number
for the UDP/STUN/TCP port object.
nit: "object" -> "objects"

[honghaiz3, 2016-10-27 17:28:58]

Description was changed from

==========
Fix another WebRTC swarming tests.

BUG=webrtc:6500
==========

to

==========
Do not rely on specific ordering on generated candidates in
TestGetAllPortsPortRange

This fixes another WebRTC swarming tests.

BUG=webrtc:6500
==========

[honghaiz3, 2016-10-27 17:31:58]

https://codereview.webrtc.org/2450983002/diff/20001/webrtc/p2p/client/basicpo...
File webrtc/p2p/client/basicportallocator_unittest.cc (right):

https://codereview.webrtc.org/2450983002/diff/20001/webrtc/p2p/client/basicpo...
webrtc/p2p/client/basicportallocator_unittest.cc:926: // Check the port number
for the UDP/STUN/TCP port object.
On 2016/10/27 16:54:08, Taylor Brandstetter wrote:
> nit: "object" -> "objects"

Done.

[honghaiz3, 2016-10-27 17:32:06]

The CQ bit was checked by honghaiz@webrtc.org

[honghaiz3, 2016-10-27 17:32:06]

The patchset sent to the CQ was uploaded after l-g-t-m from deadbeef@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2450983002/#ps40001 (title:
"Address comments")

[commit-bot: I haz the power, 2016-10-27 17:32:17]

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, 2016-10-27 18:44:19]

Description was changed from

==========
Do not rely on specific ordering on generated candidates in
TestGetAllPortsPortRange

This fixes another WebRTC swarming tests.

BUG=webrtc:6500
==========

to

==========
Do not rely on specific ordering on generated candidates in
TestGetAllPortsPortRange

This fixes another WebRTC swarming tests.

BUG=webrtc:6500
==========

[commit-bot: I haz the power, 2016-10-27 18:44:20]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-10-27 19:01:47]

Description was changed from

==========
Do not rely on specific ordering on generated candidates in
TestGetAllPortsPortRange

This fixes another WebRTC swarming tests.

BUG=webrtc:6500
==========

to

==========
Do not rely on specific ordering on generated candidates in
TestGetAllPortsPortRange

This fixes another WebRTC swarming tests.

BUG=webrtc:6500

Committed: https://crrev.com/a73df559d63b74acc1aeb8d9a4a8d26af6e6c6db
Cr-Commit-Position: refs/heads/master@{#14804}
==========

[commit-bot: I haz the power, 2016-10-27 19:01:48]

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