[Bug 28097] New: surroundContents should check host-inclusive

https://www.w3.org/Bugs/Public/show_bug.cgi?id=28097

            Bug ID: 28097
           Summary: surroundContents should check host-inclusive
           Product: WebAppsWG
           Version: unspecified
          Hardware: PC
                OS: Windows NT
            Status: NEW
          Severity: normal
          Priority: P2
         Component: DOM
          Assignee: [email protected]
          Reporter: [email protected]
        QA Contact: [email protected]
                CC: [email protected], [email protected]

Range.surroundContents() method[1] checks "partially-contained"[2] pre-check
without using host-inclusive, but then in later step when to append[3], its
pre-check is host-inclusive.

This should be consistent.

Propose to change "partially contained" to use "host-including inclusive
ancestor".

[1] http://w3c.github.io/dom/#dom-range-surroundcontents
    https://dom.spec.whatwg.org/#dom-range-surroundcontents
[2] http://w3c.github.io/dom/#partially-contained
    https://dom.spec.whatwg.org/#dom-range-surroundcontents
[3] http://w3c.github.io/dom/#concept-node-append
    https://dom.spec.whatwg.org/#concept-node-append
[4] http://w3c.github.io/dom/#partially-contained
    https://dom.spec.whatwg.org/#partially-contained

-- 
You are receiving this mail because:
You are on the CC list for the bug.