[Webkit-unassigned] [Bug 36794] Teach RedirectScheduler about URLString

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=36794


Darin Fisher (:fishd, Google) <fishd at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |brettw at chromium.org




--- Comment #14 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2010-04-02 21:04:21 PST ---
I should add, that I also find Maciej's idea interesting.  To summarize:

1- URLString contains a canonical, absolute URL string.
2- ParsedURL is constructed when you need to access/replace components from an
URLString.

Constructing an URLString from a String would require instantiating a ParsedURL
so that canonicalization could be done.  Constructing a ParsedURL from an
URLString would be cheap because ParsedURL could assume the input is already
canonical.  In such a world, KURL disappears in favor of either URLString or
ParsedURL.

This proposal might result in some hot-spots in the code that lead to us
passing around ParsedURL objects, but I suspect those will be the exception to
the norm.

Reducing the memory size of KURL so that it can be used in more places is
probably a fairly low cost and incremental change to the code base, but it may
be unnecessary work given Maciej's idea above.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.