Bug 17509 - Acid3 expects different exceptions for surroundContents calls involving comment nodes (affects Acid3 test 11)
Summary: Acid3 expects different exceptions for surroundContents calls involving comme...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Maciej Stachowiak
URL:
Keywords:
Depends on:
Blocks: Acid3
  Show dependency tree / graph
 
Reported: 2008-02-24 01:01 PST by Darin Adler
Modified: 2019-05-02 16:25 PDT (History)
2 users (show)

See Also:

Attachments
the fix (7.65 KB, patch)
2008-03-16 20:26 PDT, Maciej Stachowiak 		darin: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Adler 2008-02-24 01:01:21 PST 
This looks like low-hanging fruit to me. Should be easy to fix.
Comment 1 Darin Adler 2008-02-25 15:57:00 PST 
I started working on this and I think it's not as easy as I initially thought. The problem is that we need to check all the children that will be put inside the new element to see if they are types that are allowed. The code isn't structured to make that easy. It still looks *relatively* straightforward, but I would no longer say easy.
Comment 2 Maciej Stachowiak 2008-03-16 20:26:36 PDT 
Created attachment 19822 [details]
the fix
Comment 3 Darin Adler 2008-03-16 20:31:49 PDT 
Comment on attachment 19822 [details]
the fix

r=me
Comment 5 Lucas Forschler 2019-02-06 09:03:17 PST 
Mass moving XML DOM bugs to the "DOM" Component.