[webkit-reviews] review requested: [Bug 5476] Dynamically adding <link> for CSS style sheet outside <head> fails to load style sheet : [Attachment 7080] Proposed patch

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Wed Mar 15 01:45:26 PST 2006


Tim Omernick <timo at apple.com> has asked  for review:
Bug 5476: Dynamically adding <link> for CSS style sheet outside <head> fails to
load style sheet
http://bugzilla.opendarwin.org/show_bug.cgi?id=5476

Attachment 7080: Proposed patch
http://bugzilla.opendarwin.org/attachment.cgi?id=7080&action=edit

------- Additional Comments from Tim Omernick <timo at apple.com>
I've attached a patch which passes the test case and layout test, and does not
regress any of our layout tests.

I'm not sure if this is the best way to fix this.  Would it be a performance
problem to deepen our search for <link> elements in
DocumentImpl::recalcStyleSelector()?



More information about the webkit-reviews mailing list