<?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>14843</bug_id>
          
          <creation_ts>2011-11-16 18:01:06 +0000</creation_ts>
          <short_desc>Range interface uses enum, which doesn&apos;t exist in WebIDL</short_desc>
          <delta_ts>2011-12-10 11:27:47 +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</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#interface-range</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="Aryeh Gregor">ayg</reporter>
          <assigned_to name="Anne">annevk</assigned_to>
          <cc>annevk</cc>
    
    <cc>cam</cc>
    
    <cc>dom</cc>
    
    <cc>mike</cc>
    
    <cc>Ms2ger</cc>
    
    <cc>public-script-coord</cc>
    
    <cc>robin</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>60077</commentid>
    <comment_count>0</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2011-11-16 18:01:06 +0000</bug_when>
    <thetext>enum CompareHow : unsigned short {
    START_TO_START = 0,
    START_TO_END   = 1,
    END_TO_END     = 2,
    END_TO_START   = 3
  };

This doesn&apos;t appear to mean anything in current WebIDL, unless I&apos;m missing something as usual.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60078</commentid>
    <comment_count>1</comment_count>
    <who name="Ms2ger">Ms2ger</who>
    <bug_when>2011-11-16 18:13:24 +0000</bug_when>
    <thetext>Not missing anything, I just haven&apos;t got around to asking heycam to spec it. The idea is that WebIDL throws when passing an invalid value to compareBoundaryPoints, rather than having the check in prose.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60079</commentid>
    <comment_count>2</comment_count>
    <who name="Anne">annevk</who>
    <bug_when>2011-11-16 18:28:17 +0000</bug_when>
    <thetext>We might want to just do this one ourselves. Going forward I think we only want to have and allow string enums.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60084</commentid>
    <comment_count>3</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2011-11-16 21:01:46 +0000</bug_when>
    <thetext>This is causing the new WebIDL test framework to fail to parse the Range interface declaration, so I had to comment it out:

http://w3c-test.org/webapps/DOMCore/tests/submissions/AryehGregor/interfaces.html

We&apos;d want to update WebIDL before we could ask the maintainer of the library we&apos;re using to update that.  I could comment out just this chunk of the IDL if necessary, of course, but it would be nicer if we could parse it as-is.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60094</commentid>
    <comment_count>4</comment_count>
    <who name="Robin Berjon">robin</who>
    <bug_when>2011-11-17 10:36:55 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; We&apos;d want to update WebIDL before we could ask the maintainer of the library
&gt; we&apos;re using to update that.  I could comment out just this chunk of the IDL if
&gt; necessary, of course, but it would be nicer if we could parse it as-is.

I&apos;m the author of the library. Since I have only been needing it intermittently in recent months, my plan has been to wait for WebIDL to actually stabilise before doing a complete update. That being said, several people have needed changes to match WebIDL&apos;s fluctuations and simply patched it directly (cf https://github.com/darobin/webidl.js).

If you&apos;re really stuck and need this update though, email me and I&apos;ll try to fit it in.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60128</commentid>
    <comment_count>5</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2011-11-18 00:04:31 +0000</bug_when>
    <thetext>Thanks!  There&apos;s no change to make here, because WebIDL isn&apos;t updated yet.  DOM4&apos;s behavior here is currently just not defined.  As I try parsing more specs&apos; IDL blocks, I&apos;ll figure out what missing features are actually causing problems, although for the time being I&apos;ll just comment out the problematic bits.

As far as this bug goes, though: DOM4 as it stands just doesn&apos;t define what compareBoundaryPoints is supposed to do.  If we&apos;re going to update WebIDL, or even add the definition to DOM4, okay -- but unless someone is planning to do that really soon, we should throw the exception from prose for now so that it&apos;s actually a precise and testable requirement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60440</commentid>
    <comment_count>6</comment_count>
    <who name="Dominique Hazael-Massieux">dom</who>
    <bug_when>2011-11-24 15:56:29 +0000</bug_when>
    <thetext>I think this is a duplicate of http://www.w3.org/Bugs/Public/show_bug.cgi?id=11451 (with the possible exception that that Bug refers to &quot;string&quot; enum only)

*** This bug has been marked as a duplicate of bug 11451 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60735</commentid>
    <comment_count>7</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2011-12-01 18:50:57 +0000</bug_when>
    <thetext>Not a duplicate.  Fixing that bug would make this one moot, but if that isn&apos;t fixed, this should be fixed separately (by removing the enum from the spec).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61285</commentid>
    <comment_count>8</comment_count>
    <who name="Cameron McCormack">cam</who>
    <bug_when>2011-12-09 06:02:25 +0000</bug_when>
    <thetext>Given we&apos;re all agreed we shouldn&apos;t be introducing new integer enumerations, let&apos;s do it manually with constants as Anne suggests in comment 2, and just leave it to bug 11451 for string enums.

Flicking back to DOM Core to remove the enum from the IDL.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61288</commentid>
    <comment_count>9</comment_count>
    <who name="Cameron McCormack">cam</who>
    <bug_when>2011-12-09 06:16:29 +0000</bug_when>
    <thetext>See also bug 12810, which I&apos;m about to WONTFIX.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61298</commentid>
    <comment_count>10</comment_count>
    <who name="Anne">annevk</who>
    <bug_when>2011-12-09 11:36:48 +0000</bug_when>
    <thetext>https://bitbucket.org/ms2ger/dom-core/changeset/ab0985b1dd7a</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61388</commentid>
    <comment_count>11</comment_count>
    <who name="Ms2ger">Ms2ger</who>
    <bug_when>2011-12-10 11:27:47 +0000</bug_when>
    <thetext>http://dvcs.w3.org/hg/domcore/rev/5da6926234b7</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>