[Webkit-unassigned] [Bug 10313] xsl:import and document() don't work in stylesheets loaded via XMLHttpRequest

[bugzilla-daemon at webkit.org]

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


Alexey Proskuryakov <ap at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |abarth at webkit.org,
                   |                            |beidson at apple.com,
                   |                            |darin at apple.com,
                   |                            |mjs at apple.com




--- Comment #25 from Alexey Proskuryakov <ap at webkit.org>  2010-05-05 09:05:01 PST ---
It would be great to have this problem finally addressed, thanks for looking
into it!

One consequence of keeping a reference to originFrame is that loading from
frameless documents breaks in two cases:
1) originFrame is navigated to a different origin, so we can no longer use its
FrameLoader;
2) originFrame is destroyed.

Is this what happens in other browsers, too?

CC'ing some people who may have more insight into consequences of sharing
originFrame in this way.

We should really be making an origin frame reference for documents created via
document.implementation.createDocument() or DOMParser.parseFromString(), too.
XMLHttpRequest is an important use case, but there is nothing unique about it.

The changes to XSLImportRule::loadSheet() look as if asynchronous loading was
turned into synchronous, which would be a bad thing. But maybe it's only the
looks, I don't remember how this function really works. A ChangeLog with a
per-function explanation of changes would help a lot.

Besides a ChangeLog, a final patch would need a lot of test cases to go in -
both for things that now work, and for known problems that are prevented by
clever design. Please see <http://webkit.org/coding/contributing.html> for all
the details.

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