Bug 154627

Summary: Align our implementation of Range.createContextualFragment with the specification
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: DOMAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, darin, esprehn+autocc, kangil.han, rniwa, russell
Priority: P2 Keywords: WebExposed
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
URL: https://w3c.github.io/DOM-Parsing/#widl-Range-createContextualFragment-DocumentFragment-DOMString-fragment
Attachments:
Description Flags
Patch
none
Patch none

Chris Dumez
Reported 2016-02-23 19:02:45 PST
Align our implementation of Range.createContextualFragment with the specification: - https://w3c.github.io/DOM-Parsing/#widl-Range-createContextualFragment-DocumentFragment-DOMString-fragment
Attachments
Patch (19.94 KB, patch)
2016-02-23 20:18 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (19.85 KB, patch)
2016-02-23 20:20 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2016-02-23 19:04:04 PST
Chrome already seems to match the specification.
Chris Dumez
Comment 2 2016-02-23 20:07:55 PST
*** Bug 154594 has been marked as a duplicate of this bug. ***
Chris Dumez
Comment 3 2016-02-23 20:18:18 PST
Created attachment 272079 [details] Patch
Chris Dumez
Comment 4 2016-02-23 20:20:48 PST
Created attachment 272080 [details] Patch
WebKit Commit Bot
Comment 5 2016-02-23 21:15:39 PST
Comment on attachment 272080 [details] Patch Clearing flags on attachment: 272080 Committed r197012: <http://trac.webkit.org/changeset/197012>
WebKit Commit Bot
Comment 6 2016-02-23 21:15:43 PST
All reviewed patches have been landed. Closing bug.
besworks
Comment 7 2016-02-24 10:28:15 PST
(In reply to comment #6) > All reviewed patches have been landed. Closing bug. Your patches have solved the issue I was having perfectly. Thanks for the quick turnaround.
Chris Dumez
Comment 8 2016-02-24 10:29:36 PST
(In reply to comment #7) > (In reply to comment #6) > > All reviewed patches have been landed. Closing bug. > > Your patches have solved the issue I was having perfectly. Thanks for the > quick turnaround. Sure, no problem. Next time, please cc me on your newly filed DOM bugs.
Note You need to log in before you can comment on or make changes to this bug.