Don't allow a leading plus in ports for proxy bypass rules.

Don't allow a leading plus in ports for proxy bypass rules.

For instance will reject the rule "*.org:+443" whereas before it would be accepted.

BUG=596523, 596570
Committed: https://crrev.com/55fa65c5ce1c15d8a492d6876b2324fd576a83a4
Cr-Commit-Position: refs/heads/master@{#383111}

[eroman, 2016-03-23 22:39:49]

The CQ bit was checked by eroman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-23 22:40:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1831653002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1831653002/1

[eroman, 2016-03-23 22:41:52]

eroman@chromium.org changed reviewers:
+ mmenke@chromium.org

[eroman, 2016-03-23 22:41:53]

https://codereview.chromium.org/1831653002/diff/1/net/proxy/proxy_bypass_rule...
File net/proxy/proxy_bypass_rules.cc (left):

https://codereview.chromium.org/1831653002/diff/1/net/proxy/proxy_bypass_rule...
net/proxy/proxy_bypass_rules.cc:320: host = raw;
This removal is not strictly part of this change.
But it didn't make sense when reading this so I removed it (unreferenced)

[eroman, 2016-03-23 22:43:19]

Description was changed from

==========
Don't allow a leading plus in ports for proxy bypass rules.

For instance will reject the rule "*.org:+443" whereas before it would be
accepted.

BUG=596570
==========

to

==========
Don't allow a leading plus in ports for proxy bypass rules.

For instance will reject the rule "*.org:+443" whereas before it would be
accepted.

BUG=596523, 596570
==========

[commit-bot: I haz the power, 2016-03-24 00:09:56]

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

[commit-bot: I haz the power, 2016-03-24 00:09:57]

Dry run: This issue passed the CQ dry run.

[mmenke, 2016-03-24 15:43:35]

On 2016/03/24 00:09:57, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

LGTM

[eroman, 2016-03-24 17:44:57]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2016-03-24 17:45:15]

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

[commit-bot: I haz the power, 2016-03-24 18:57:57]

Description was changed from

==========
Don't allow a leading plus in ports for proxy bypass rules.

For instance will reject the rule "*.org:+443" whereas before it would be
accepted.

BUG=596523, 596570
==========

to

==========
Don't allow a leading plus in ports for proxy bypass rules.

For instance will reject the rule "*.org:+443" whereas before it would be
accepted.

BUG=596523, 596570
==========

[commit-bot: I haz the power, 2016-03-24 18:57:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-24 19:00:23]

Description was changed from

==========
Don't allow a leading plus in ports for proxy bypass rules.

For instance will reject the rule "*.org:+443" whereas before it would be
accepted.

BUG=596523, 596570
==========

to

==========
Don't allow a leading plus in ports for proxy bypass rules.

For instance will reject the rule "*.org:+443" whereas before it would be
accepted.

BUG=596523, 596570

Committed: https://crrev.com/55fa65c5ce1c15d8a492d6876b2324fd576a83a4
Cr-Commit-Position: refs/heads/master@{#383111}
==========

[commit-bot: I haz the power, 2016-03-24 19:00:24]

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