ParseCandidate(): Refactor to fix memcheck false positive.

ParseCandidate(): Refactor to fix memcheck false positive.
Also make supported protocols explicit in check.

Fix inconsistency where TLS_PROTOCOL_NAME was not exported.

BUG=webrtc:6885
Committed: https://crrev.com/b68cc75f1959218970df168b4c1b6e724692ac39
Cr-Commit-Position: refs/heads/master@{#15577}

[hnsl1, 2016-12-13 17:47:07]

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

[commit-bot: I haz the power, 2016-12-13 17:47:17]

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

[hnsl1, 2016-12-13 17:49:35]

hnsl@webrtc.org changed reviewers:
+ deadbeef@webrtc.org, pthatcher@webrtc.org

[hnsl1, 2016-12-13 17:49:35]

Please review this CL which fixes a false positive regression in
https://codereview.webrtc.org/2568833002/

[hnsl1, 2016-12-13 17:53:52]

Description was changed from

==========
ParseCandidate(): Refactor and improve protocol check to work around
memcheck false positive and make check more explicit.

Fix inconsistency where TLS_PROTOCOL_NAME was not exported.

BUG=6885
==========

to

==========
ParseCandidate(): Refactor to fix memcheck false positive.
Also make supported protocols explicit in check.

Fix inconsistency where TLS_PROTOCOL_NAME was not exported.

BUG=webrtc:6885
==========

[commit-bot: I haz the power, 2016-12-13 18:03:07]

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

[commit-bot: I haz the power, 2016-12-13 18:03:07]

Dry run: This issue passed the CQ dry run.

[Taylor Brandstetter, 2016-12-13 18:25:26]

lgtm. I had no idea this kind of thing could trip up memcheck; good to know.

[hnsl1, 2016-12-13 18:28:07]

The CQ bit was checked by hnsl@webrtc.org

[commit-bot: I haz the power, 2016-12-13 18:28:15]

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-12-13 18:33:42]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481653687532480, "parent_rev":
"ee31d5cf690bc03b83b34bb0e779ee08d12547cb", "commit_rev":
"01c285db8aa5ccaa578602a045f0d72e4ed59c34"}

[commit-bot: I haz the power, 2016-12-13 18:33:45]

Description was changed from

==========
ParseCandidate(): Refactor to fix memcheck false positive.
Also make supported protocols explicit in check.

Fix inconsistency where TLS_PROTOCOL_NAME was not exported.

BUG=webrtc:6885
==========

to

==========
ParseCandidate(): Refactor to fix memcheck false positive.
Also make supported protocols explicit in check.

Fix inconsistency where TLS_PROTOCOL_NAME was not exported.

BUG=webrtc:6885

Review-Url: https://codereview.webrtc.org/2570803003
==========

[commit-bot: I haz the power, 2016-12-13 18:33:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-13 18:33:53]

Description was changed from

==========
ParseCandidate(): Refactor to fix memcheck false positive.
Also make supported protocols explicit in check.

Fix inconsistency where TLS_PROTOCOL_NAME was not exported.

BUG=webrtc:6885

Review-Url: https://codereview.webrtc.org/2570803003
==========

to

==========
ParseCandidate(): Refactor to fix memcheck false positive.
Also make supported protocols explicit in check.

Fix inconsistency where TLS_PROTOCOL_NAME was not exported.

BUG=webrtc:6885

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

[commit-bot: I haz the power, 2016-12-13 18:33:54]

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