Add more test cases for HttpUtil::PathForRequest() and SpecForRequest()

Add more test cases for HttpUtil::PathForRequest() and SpecForRequest()

https://codereview.chromium.org/68213017/ made them able to handle
WebSocket schemes. Add tests to check that.

Bonus: some more non-WebSocket cases to increase coverage.

BUG=303568
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=237941

[tyoshino (SeeGerritForStatus), 2013-11-15 04:19:53]

Please ignore http_util_icu.cc change. I'll resolve it after your change is
committed.

[Adam Rice, 2013-11-15 08:01:44]

https://codereview.chromium.org/73633002/diff/1/net/http/http_util_unittest.cc
File net/http/http_util_unittest.cc (right):

https://codereview.chromium.org/73633002/diff/1/net/http/http_util_unittest.c...
net/http/http_util_unittest.cc:641: // ftp scheme
I would prefer to have this as a separate test case, since it doesn't need to be
in this test case. You could call it SpecForFtpRequest or something similar.

[tyoshino (SeeGerritForStatus), 2013-11-15 13:35:49]

https://codereview.chromium.org/73633002/diff/1/net/http/http_util_unittest.cc
File net/http/http_util_unittest.cc (right):

https://codereview.chromium.org/73633002/diff/1/net/http/http_util_unittest.c...
net/http/http_util_unittest.cc:641: // ftp scheme
On 2013/11/15 08:01:44, Adam Rice wrote:
> I would prefer to have this as a separate test case, since it doesn't need to
be
> in this test case. You could call it SpecForFtpRequest or something similar.

Done.

[Adam Rice, 2013-11-18 06:43:42]

lgtm

[tyoshino (SeeGerritForStatus), 2013-11-29 04:42:17]

+szym

[szym, 2013-11-29 11:08:24]

lgtm

[commit-bot: I haz the power, 2013-11-29 17:39:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tyoshino@chromium.org/73633002/150001

[commit-bot: I haz the power, 2013-11-29 19:59:32]

Change committed as 237941