Simplify protocol name and port comparison in KnwonPorts.cpp

Simplify protocol name and port comparison in KnwonPorts.cpp

Locking on hashmap initialization may causes problem when constructing
referer. Change it to a simpler approach.

(Original patch https://codereview.chromium.org/895793003/
created by Ian Wen (Ianwen@) and added test)

BUG=373141, 441605
TEST=KnownPortsTest.IsDefaultPortForProtocol

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194007

[kinuko, 2015-04-20 06:25:58]

kinuko@chromium.org changed reviewers:
+ ianwen@chromium.org, mkwst@chromium.org

[Mike West, 2015-04-20 07:06:34]

LGTM.

Only thing I note here is that the function used to be case-insensitive. It's
fine to change that if we don't need it, but please add some tests that make
sure it breaks if given uppercase letters.

[kinuko, 2015-04-20 07:32:54]

On 2015/04/20 07:06:34, Mike West wrote:
> LGTM.
> 
> Only thing I note here is that the function used to be case-insensitive. It's
> fine to change that if we don't need it, but please add some tests that make
> sure it breaks if given uppercase letters.

Thanks! Good point, added test cases for that.

[kinuko, 2015-04-20 07:32:58]

The CQ bit was checked by kinuko@chromium.org

[kinuko, 2015-04-20 07:32:58]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org
Link to the patchset: https://codereview.chromium.org/1074913004/#ps20001
(title: "added upper-case test case")

[commit-bot: I haz the power, 2015-04-20 07:33:15]

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

[commit-bot: I haz the power, 2015-04-20 09:18:31]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194007