Bug 17622

Summary: insertAdjacentHTML and insertAdjacentText are not implemented although insertAdjacentElement is.
Product: WebKit Reporter: Kevin McCullough <kmccullough>
Component: DOMAssignee: Sam Weinig <sam>
Status: RESOLVED FIXED    
Severity: Normal CC: caugusto, gavin.sharp
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 8947    
Attachments:
Description Flags
WIP
none
updated with tests none

Kevin McCullough
Reported 2008-02-29 17:36:30 PST
These are three IE functions that we would like to implement. Currently we only have insertAdjacentElement but not insertAdjacentHTML or insertAdjacentText, and it just so happens that http://www.openwebware.com/products/openwysiwyg/demo.shtml checks for the existence of insertAdjacentElement and if found, assumes insertAdjacentHTML and insertAdjacentText exist too. So the site fails because the functions are missing.
Attachments
WIP (12.23 KB, patch)
2008-03-01 15:06 PST, Sam Weinig 		no flags
DetailsFormatted DiffDiff
updated with tests (25.25 KB, patch)
2008-06-14 18:24 PDT, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Kevin McCullough
Comment 1 2008-02-29 17:38:53 PST
<rdar://problem/5775192>
Mark Rowe (bdash)
Comment 2 2008-02-29 17:49:05 PST
*** Bug 17624 has been marked as a duplicate of this bug. ***
Mark Rowe (bdash)
Comment 3 2008-02-29 17:49:18 PST
*** Bug 17623 has been marked as a duplicate of this bug. ***
Sam Weinig
Comment 4 2008-03-01 15:06:08 PST
Created attachment 19474 [details] WIP This is mostly done, I just need to make some test cases.
Sam Weinig
Comment 5 2008-03-01 15:07:34 PST
I moved insertAdjacentElement into HTMLElement to match Opera and IE.
mitz
Comment 6 2008-03-01 15:26:58 PST
*** Bug 7633 has been marked as a duplicate of this bug. ***
Sam Weinig
Comment 7 2008-06-14 18:24:33 PDT
Created attachment 21704 [details] updated with tests
Sam Weinig
Comment 8 2008-06-14 19:06:00 PDT
Mitz reviewed an updated version of the patch. Fix landed in r34543.
Note You need to log in before you can comment on or make changes to this bug.