[Webkit-unassigned] [Bug 51941] refactor HTMLLinkElement to allow Link header implementation

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 7 13:24:46 PST 2011


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





--- Comment #29 from Alexey Proskuryakov <ap at webkit.org>  2011-01-07 13:24:46 PST ---
> I move the clearOwnerNode() call to the proper destructor.

Sorry, I got fooled by a diff in HTMLLinkElement::~HTMLLinkElement().

I think that this patch needs further scaling down in the direction outlined above, but I can't afford the time for a line by line review right now, sorry.

However, RefPtr<CSSStyleSheet> m_sheet should definitely stay in HTMLLinkElement.

>> Firing beforeload is probably one of those, but it means that loads initiated from HTTP headers need
>> to be delayed until after a beforeload handler has a chance to be installed. Is that something that's 
>> going to be implemented?
>
> Probably not.

This makes me quite suspicious about adding even rel=prefetch. Beforeload exists to let one block or change all subresource loads.

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



More information about the webkit-unassigned mailing list