net: use IsSecureScheme rather than matching "https".

net: use IsSecureScheme rather than matching "https".

My r202927 matched against a scheme of "https" in order to be consistent with
the other users in the same file. But it appears that IsSecureScheme is the
better answer throughout the file.

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

[wtc, 2013-06-11 17:49:32]

Patch set 1 LGTM.

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

https://codereview.chromium.org/16776005/diff/1/net/http/http_stream_parser.c...
net/http/http_stream_parser.cc:552: return ERR_HEADERS_TRUNCATED;

Nit: Can you also fix the name of the error code?

ERR_HEADERS_TRUNCATED => ERR_RESPONSE_HEADERS_TRUNCATED

[commit-bot: I haz the power, 2013-06-11 20:58:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/agl@chromium.org/16776005/5001

[agl, 2013-06-11 20:58:39]

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

https://codereview.chromium.org/16776005/diff/1/net/http/http_stream_parser.c...
net/http/http_stream_parser.cc:552: return ERR_HEADERS_TRUNCATED;
On 2013/06/11 17:49:33, wtc wrote:
> 
> Nit: Can you also fix the name of the error code?
> 
> ERR_HEADERS_TRUNCATED => ERR_RESPONSE_HEADERS_TRUNCATED

Done.

[commit-bot: I haz the power, 2013-06-11 23:20:54]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-06-12 19:01:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/agl@chromium.org/16776005/5001

[commit-bot: I haz the power, 2013-06-12 19:43:04]

Change committed as 205898