[Webkit-unassigned] [Bug 164907] Add URL::hostAndPort()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Nov 18 08:32:01 PST 2016


Alex Christensen <achristensen at apple.com> changed:

           What    |Removed                     |Added
 Attachment #295141|review?                     |review+
              Flags|                            |

--- Comment #2 from Alex Christensen <achristensen at apple.com> ---
Comment on attachment 295141
  --> https://bugs.webkit.org/attachment.cgi?id=295141

View in context: https://bugs.webkit.org/attachment.cgi?id=295141&action=review

> Source/WebCore/html/URLUtils.h:132
>      if (!url.port() || isDefaultPortForProtocol(url.port().value(), url.protocol()))
>          return url.host();

We can also remove this.  The new URLParser will always remove the port if it's the default port for the protocol.
Same with the check for hostEnd and pathStart.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20161118/25156903/attachment.html>

More information about the webkit-unassigned mailing list