Bug 5725 - setAttributeNS with null namespace sets namespace to string containing 'null'
Summary: setAttributeNS with null namespace sets namespace to string containing 'null'
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Anders Carlsson
URL:
Keywords:
: 5724 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-11-13 04:05 PST by Anders Carlsson
Modified: 2019-02-06 09:04 PST (History)
1 user (show)

See Also:


Attachments
Patch (7.68 KB, patch)
2005-11-13 04:06 PST, Anders Carlsson
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Carlsson 2005-11-13 04:05:28 PST
This is actually three bugs which are related:

Due to a typo, the code in isEqualNode doesn't check if the attributes differ.

When fixing this, one of the DOM3 tests started failing. This is because the tests load documents by 
loading them in a frame/iframe (I'm not sure) and then cloning the document manually. In this code, 
there's a call to setAttributeNS which sets the namespaceURI to 'null', so it won't be the same as the 
original.

When fixing the setAttributeNS bug, run-webkit-tests started crashing. This is because a call to 
setAttributeNS to set the src on a <script> element would now result in the url being loaded (That 
wasn't the case before when the namespaceURI always was null) and we don't support loading external 
scripts from viewless documents (documents created by document.implementation.createDocument)
While we do want to load external scripts later (for xbl according to hyatt) I just made it return 
immediatly when there's no part available.
Comment 1 Anders Carlsson 2005-11-13 04:06:32 PST
Created attachment 4678 [details]
Patch

I don't really like the name valueToStringWithNullCheck but I couldn't come up
with any better...
Comment 2 Anders Carlsson 2005-11-13 04:11:57 PST
*** Bug 5724 has been marked as a duplicate of this bug. ***
Comment 3 Eric Seidel (no email) 2005-11-13 10:16:40 PST
Comment on attachment 4678 [details]
Patch

We talked about this on IRC.  Anders is going to file a bug about scripts not
being evaluated on viewless documents.	Otherwise this looks fine to me.
Comment 4 Lucas Forschler 2019-02-06 09:04:12 PST
Mass moving XML DOM bugs to the "DOM" Component.