web.http.Request.setHost ignores the port when setting the host header

[armooo]

While working on 3845 (which ended up looking like a dup of 2648) I noticed that web.http.Request.setHost is ignoring the port number when setting the host header.

[armooo]

Ignore the first attachment which is the wrong file.

[exarkun]

Thank you for the patch, armooo. This looks great. One minor point, we're preferring assertEquals over assertEqual for new tests, and we use camelCase instead of under_scores for things like the host_header local in setHost. Those are trivial though, I'll adjust the patch and then apply it. Thanks again!

[exarkun]

(In [30876]) Apply patch to cause Request.setHost to set a header which includes port information if appropriate.

Author: armooo Reviewer: exarkun Fixes: #4918

If the port passed to Request.setHost isn't the default for the scheme, include it in the header added to Request.requestHeaders to more closely imitate the header which would be present if the client were really issuing requests to the specified host and port.