Bug 4493

Summary: add qualifiedName checking for empty string
Product: WebKit Reporter: Darin Adler <darin>
Component: DOMAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: adele, andersca, cdumez, curt.arnold, mjs
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Bug Depends on:    
Bug Blocks: 4565    
Attachments:
Description Flags
adds code that knows that an empty qualified name is not valid sullivan: review+

Darin Adler
Reported 2005-08-17 08:59:26 PDT
Attachments
adds code that knows that an empty qualified name is not valid (4.50 KB, patch)
2005-08-17 09:00 PDT, Darin Adler
sullivan: review+
Darin Adler
Comment 1 2005-08-17 09:00:48 PDT
Created attachment 3438 [details] adds code that knows that an empty qualified name is not valid
Darin Adler
Comment 2 2005-08-17 11:42:03 PDT
Comment on attachment 3438 [details] adds code that knows that an empty qualified name is not valid Adele and Maciej are looking into this. The DOM specification clearly doesn't call for INVALID_CHARACTER_ERR in the case of an empty string, so we're trying to learn more about the conference call where someone decided this was correct behavior.
Darin Adler
Comment 3 2005-09-03 12:53:23 PDT
This bug affects these tests: dom/html/level2/core/createDocument08.html dom/html/level2/core/createDocumentType04.html
Darin Adler
Comment 4 2005-09-05 15:23:48 PDT
Looks like this is definitely correct for DOM Level 3 and arguably OK for Level 2, so we're going to make the change.
Maciej Stachowiak
Comment 5 2005-09-05 22:27:58 PDT
This behavior not even arguably ok for level 2, barring a possible future change to the level 2 spec. It likely doesn't matter for any real-world cases, but it seems silly to make a change for a test that isn't testing what it says it is.
Darin Adler
Comment 6 2005-09-06 09:25:30 PDT
OK. Maciej, you're welcome to change this back, or resolve the problem in whatever other way you see fit. Since I didn't get any comment from you on this for a couple of weeks, I mistakenly thought you agreed with Curt that this should just be changed in our code.
Lucas Forschler
Comment 7 2019-02-06 09:02:52 PST
Mass moving XML DOM bugs to the "DOM" Component.
Note You need to log in before you can comment on or make changes to this bug.