Bug 8197
| Summary: | REGRESSION: Realm status is not displayed at worldofwarcraft.com | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Edward Keech <ed.keech> |
| Component: | XML | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, eric |
| Priority: | P1 | Keywords: | Regression |
| Version: | 420+ | ||
| Hardware: | Mac | ||
| OS: | OS X 10.4 | ||
| URL: | http://www.worldofwarcraft.com/realmstatus/ | ||
| Bug Depends on: | 7838 | ||
| Bug Blocks: | |||
Edward Keech
Doesn't display the table where the server's are. Safari works fine.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Darin Adler
Alexey changed the component to XML, because the error that prevents the page from working is "xslDoc.load" not working. I believe that this worked before because we didn't have XSLTProcessor, and it's failing now because the document returned by XSLTProcessor.getDomDocument does not have a load function. Here's the code:
tableProcessor = new XSLTProcessor();
var xslDoc = Sarissa.getDomDocument();
xslDoc.async = false;
xslDoc.load('/realmstatus/realmstatus.xsl');
tableProcessor.importStylesheet(xslDoc);
Dave Hyatt
Isn't the load method from DOM 3 Load and Save? I don't think we've ever supported it.
Dave Hyatt
Or is this their own JS document wrapper and not a real DOM document?
Alexey Proskuryakov
Sarissa provides its own load(), but only if window.XMLDocument is defined, see bug 7838.
else{ /* end IE initialization, try to deal with real browsers now ;-) */
if(_SARISSA_HAS_DOM_CREATE_DOCUMENT){
if(window.XMLDocument){
...
XMLDocument.prototype.load = function(sURI) {
Alexey Proskuryakov
The patch in bug 7838 did fix this. The page is still not 100% OK, though, see bug 8852.