<?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>12288</bug_id>
          
          <creation_ts>2011-03-11 15:18:12 +0000</creation_ts>
          <short_desc>select.size should be limited to non-negative numbers instead of non-negative numbers greater than zero</short_desc>
          <delta_ts>2011-08-15 19:34:24 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>HTML WG</product>
          <component>LC1 HTML5 spec</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></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="Mounir Lamouri">mounir</reporter>
          <assigned_to name="Ian &apos;Hixie&apos; Hickson">ian</assigned_to>
          <cc>ayg</cc>
    
    <cc>bugs</cc>
    
    <cc>contributor</cc>
    
    <cc>ian</cc>
    
    <cc>jonas</cc>
    
    <cc>mike</cc>
    
    <cc>public-html-admin</cc>
    
    <cc>public-html-wg-issue-tracking</cc>
          
          <qa_contact name="HTML WG Bugzilla archive list">public-html-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>46630</commentid>
    <comment_count>0</comment_count>
    <who name="Mounir Lamouri">mounir</who>
    <bug_when>2011-03-11 15:18:12 +0000</bug_when>
    <thetext>Gecko has select.size limited to non-negative numbers greater than zero since nearly 10 months now but we recently got three bug reports that let us think this change wasn&apos;t web compatible.

Currently, select.size works this way for other vendors:
- no limitation: Firefox 3.6, Chrome 11, Opera 11 and Webkit trunk ;
- limited to non-negative numbers: IE6 and IE9 (I assume 7 and 8 too) ;
- limited to non-negative numbers greater than zero: Firefox 4.

Given that the current specs state isn&apos;t web compatible and IE6/9 have select.size limited to non-negative numbers, we can probably have select.size limited to non-negative numbers instead of non-negative numbers greater than zero.

For information. the tree Gecko bugs:
https://bugzilla.mozilla.org/show_bug.cgi?id=639175 (issue with Yahoo Answers)
https://bugzilla.mozilla.org/show_bug.cgi?id=640636 (issue with jQuery)
https://bugzilla.mozilla.org/show_bug.cgi?id=640800 (no website involved)

It goes without saying that these three bugs were involving select.size=0.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>46658</commentid>
    <comment_count>1</comment_count>
    <who name="Mounir Lamouri">mounir</who>
    <bug_when>2011-03-14 02:27:02 +0000</bug_when>
    <thetext>Even if I don&apos;t really think this bug need more arguments, select.size is currently in a situation where, this raises an exception:
var select = document.createElement(&apos;select&apos;);
select.size = select.size;

Given that 0 is the default value.
I believe this is seriously wrong...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>48295</commentid>
    <comment_count>2</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2011-05-07 19:17:26 +0000</bug_when>
    <thetext>EDITOR&apos;S RESPONSE: This is an Editor&apos;s Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document:
   http://dev.w3.org/html5/decision-policy/decision-policy.html

Status: Accepted
Change Description: see diff given below
Rationale: Concurred with reporter&apos;s comments.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>48296</commentid>
    <comment_count>3</comment_count>
    <who name="">contributor</who>
    <bug_when>2011-05-07 19:17:54 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r6119.
Check-in comment: select.size has odd legacy behaviour required for compat
http://html5.org/tools/web-apps-tracker?from=6118&amp;to=6119</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>48450</commentid>
    <comment_count>4</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2011-05-10 17:42:05 +0000</bug_when>
    <thetext>This introduces a contradiction.  In the IDL, &quot;size&quot; is an unsigned long, but the prose says it&apos;s limited to only non-negative numbers.  The prose for &quot;limited to only non-negative numbers&quot; is defined only for long, not unsigned long: &quot;If a reflecting IDL attribute has a signed integer type (long) that is limited to only non-negative numbers . . .&quot;

You probably meant to change the IDL type for &quot;size&quot; from &quot;unsigned long&quot; to &quot;long&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>49286</commentid>
    <comment_count>5</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2011-06-07 21:33:11 +0000</bug_when>
    <thetext>*** Bug 12899 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>53924</commentid>
    <comment_count>6</comment_count>
    <who name="Michael[tm] Smith">mike</who>
    <bug_when>2011-08-04 05:34:31 +0000</bug_when>
    <thetext>mass-move component to LC1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>55022</commentid>
    <comment_count>7</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2011-08-15 04:21:05 +0000</bug_when>
    <thetext>EDITOR&apos;S RESPONSE: This is an Editor&apos;s Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document:
   http://dev.w3.org/html5/decision-policy/decision-policy.html

Status: Accepted
Change Description: see diff given below
Rationale: Yeah, that wasn&apos;t right. But the fix in comment 4 isn&apos;t quite right either. :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>55023</commentid>
    <comment_count>8</comment_count>
    <who name="">contributor</who>
    <bug_when>2011-08-15 04:21:24 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r6455.
Check-in comment: Actually fix r6119 properly.
http://html5.org/tools/web-apps-tracker?from=6454&amp;to=6455</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>55134</commentid>
    <comment_count>9</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2011-08-15 19:34:24 +0000</bug_when>
    <thetext>Looks good to me.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>