<?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>17331</bug_id>
          
          <creation_ts>2012-06-05 11:22:48 +0000</creation_ts>
          <short_desc>(AudioParamUnits): AudioParam.units undefined</short_desc>
          <delta_ts>2012-07-09 09:16:36 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>AudioWG</product>
          <component>Web Audio API</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>CLOSED</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>TBD</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Philip Jägenstedt">philipj</reporter>
          <assigned_to name="This bug has no owner yet - up for the taking">dave.null</assigned_to>
          <cc>crogers</cc>
    
    <cc>mage</cc>
    
    <cc>olivier.thereaux</cc>
    
    <cc>philipj</cc>
    
    <cc>public-audio</cc>
          
          <qa_contact name="This bug has no owner yet - up for the taking">dave.null</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>68641</commentid>
    <comment_count>0</comment_count>
    <who name="Philip Jägenstedt">philipj</who>
    <bug_when>2012-06-05 11:22:48 +0000</bug_when>
    <thetext>Audio-ISSUE-35 (AudioParamUnits): AudioParam.units undefined [Web Audio API]

http://www.w3.org/2011/audio/track/issues/35

Raised by: Philip Jägenstedt
On product: Web Audio API

It looks like this was intended to be an enumeration, but it has not been defined. What value should be returned and what is the use case?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69034</commentid>
    <comment_count>1</comment_count>
    <who name="Chris Rogers">crogers</who>
    <bug_when>2012-06-14 00:52:17 +0000</bug_when>
    <thetext>This has been removed in:
https://dvcs.w3.org/hg/audio/rev/d1fb69113f02

For now, at least, it seems better to leave this out, since the use case is not clear</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69200</commentid>
    <comment_count>2</comment_count>
    <who name="Philip Jägenstedt">philipj</who>
    <bug_when>2012-06-18 12:37:58 +0000</bug_when>
    <thetext>We obviously agree with the removal, but lots of unrelated changes are included in the diff. Are those in need of review, and in the context of what bug?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69581</commentid>
    <comment_count>3</comment_count>
    <who name="Olivier Thereaux">olivier.thereaux</who>
    <bug_when>2012-07-03 07:47:40 +0000</bug_when>
    <thetext>(In reply to comment #2)
&gt; We obviously agree with the removal, but lots of unrelated changes are included
&gt; in the diff. Are those in need of review, and in the context of what bug?

The other changes seem to be mostly examples and clarifications, so probably uncontroversial. 

Back to the topic of AudioParam.units - Chris, could you give details as to why that bit has been removed? Philip seemed to disagree with it (though not enough to reopen the bug with a counter proposal) and I&apos;d like at least for the rationale to be documented.

Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69606</commentid>
    <comment_count>4</comment_count>
    <who name="Chris Rogers">crogers</who>
    <bug_when>2012-07-03 20:30:25 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; &gt; We obviously agree with the removal, but lots of unrelated changes are included
&gt; &gt; in the diff. Are those in need of review, and in the context of what bug?
&gt; 
&gt; The other changes seem to be mostly examples and clarifications, so probably
&gt; uncontroversial. 
&gt; 
&gt; Back to the topic of AudioParam.units - Chris, could you give details as to why
&gt; that bit has been removed? Philip seemed to disagree with it (though not enough
&gt; to reopen the bug with a counter proposal) and I&apos;d like at least for the
&gt; rationale to be documented.
&gt; 
&gt; Thanks.

Up to this point, the &quot;units&quot; attribute has really just been a stub with no real implementation, and we haven&apos;t yet defined different unit types yet (such as frequency, cents, etc.).  So this is just a simplification which should not impact any content out in the field.  So far, we&apos;ve seen no urgent need, but if we later decide that this is important, then we can add it back in, defining the precise types.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69730</commentid>
    <comment_count>5</comment_count>
    <who name="Marcus Geelnard (Opera)">mage</who>
    <bug_when>2012-07-09 08:53:28 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; Back to the topic of AudioParam.units - Chris, could you give details as to why
&gt; that bit has been removed? Philip seemed to disagree with it

Just to clarify: we (me and Philip) think it&apos;s a good idea to remove AudioParam.units.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69731</commentid>
    <comment_count>6</comment_count>
    <who name="Olivier Thereaux">olivier.thereaux</who>
    <bug_when>2012-07-09 09:16:36 +0000</bug_when>
    <thetext>(In reply to comment #5)
&gt; Just to clarify: we (me and Philip) think it&apos;s a good idea to remove
&gt; AudioParam.units.

Thanks for the clarification Marcus. I see no reason to keep this issue open, then.

[CLOSED]</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>