<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>29174</bug_id>
          
          <creation_ts>2015-10-03 01:03:36 +0000</creation_ts>
          <short_desc>ProcessingInstruction node is correct (as context) for Range.createContextualFragment() method</short_desc>
          <delta_ts>2015-10-03 01:03:36 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebAppsWG</product>
          <component>DOM Parsing and Serialization</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows NT</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Arkadiusz Michalski (Spirit)">crimsteam</reporter>
          <assigned_to name="Travis Leithead [MSFT]">travil</assigned_to>
          <cc>mike</cc>
    
    <cc>www-dom</cc>
          
          <qa_contact>public-webapps-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>123464</commentid>
    <comment_count>0</comment_count>
    <who name="Arkadiusz Michalski (Spirit)">crimsteam</who>
    <bug_when>2015-10-03 01:03:36 +0000</bug_when>
    <thetext>In algo for Range.createContextualFragment() method:
http://www.w3.org/TR/DOM-Parsing/#dfn-dom-range-createcontextualfragment
ProcessingInstruction node should not be forbidden as context, DOM4 prevents only DocumentType when setting range&apos;s start/end node:
http://www.w3.org/TR/dom/#concept-range-bp-set
and browsers (Firefox, Chrome, IE11) not throw for ProcessingInstruction.
This node probably should be treat the same as Text and Comment.

BTW, will be better changing variable names to something more obvious (what whe have for other methods):
- passing argument /fragment/ to /text/ or /str/ (step 1)
- /fragment node/ to /fragment/ (step 3/4/5)
- and maybe /element/ to /context/ (like whe have for insertAdjacentHTML)
- correct IDL and green box too

One more thing, tables with Parameter/Type/Nullable/Optional are necessary? The same infos we see directly in IDL, so why repeat and unnecessarily making spec bigger?</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>