From 4f2afe9adcdfa7d77b08efbaa44f880d93f48e41 Mon Sep 17 00:00:00 2001
From: Raymond Toy <rtoy@chromium.org>
Date: Thu, 23 Jun 2016 14:39:40 -0700
Subject: [PATCH] Fix #30: allow disabling resampling for decodeAudioData

Add yet another optional dictionary parameter to decodeAudioData to
allow disabling the resampling for decodeAudioData.

Defines a new dictionary for these options.

Links and processing model updated appropriately.
---
 index.html | 45 +++++++++++++++++++++++++++++++++++++++++----
 1 file changed, 41 insertions(+), 4 deletions(-)

diff --git a/index.html b/index.html
index 0522ded6e..cbe19875d 100644
--- a/index.html
+++ b/index.html
@@ -733,7 +733,7 @@ <h2 id="BaseAudioContext">
             still be created, through <a href=
             "#widl-BaseAudioContext-createBuffer-AudioBuffer-unsigned-long-numberOfChannels-unsigned-long-length-float-sampleRate">
             createBuffer</a> or <a href=
-            "#widl-BaseAudioContext-decodeAudioData-Promise-AudioBuffer--ArrayBuffer-audioData-DecodeSuccessCallback-successCallback-DecodeErrorCallback-errorCallback">
+            "#widl-BaseAudioContext-decodeAudioData-Promise-AudioBuffer--ArrayBuffer-audioData-DecodeSuccessCallback-successCallback-DecodeErrorCallback-errorCallback-DecodeAudioDataOptions-options">
             decodeAudioData</a>.)
           </dd>
         </dl>
@@ -1179,7 +1179,7 @@ <h2 id="BaseAudioContext">
               <code>responseType</code> to <code>"arraybuffer"</code>. Audio
               file data can be in any of the formats supported by the
               <code>audio</code> element. The buffer passed to <a href=
-              "#widl-AudioContext-decodeAudioData-Promise-AudioBuffer--ArrayBuffer-audioData-DecodeSuccessCallback-successCallback-DecodeErrorCallback-errorCallback">
+              "#widl-BaseAudioContext-decodeAudioData-Promise-AudioBuffer--ArrayBuffer-audioData-DecodeSuccessCallback-successCallback-DecodeErrorCallback-errorCallback-DecodeAudioDataOptions-options">
               decodeAudioData</a> has its content-type determined by sniffing,
               as described in [[mimesniff]].
             </p>
@@ -1205,6 +1205,14 @@ <h2 id="BaseAudioContext">
                 A callback function which will be invoked if there is an error
                 decoding the audio file.
               </dd>
+              <dt>
+                optional DecodeAudioDataOptions options
+              </dt>
+              <dd>
+                Options for <a href=
+                "#widl-BaseAudioContext-decodeAudioData-Promise-AudioBuffer--ArrayBuffer-audioData-DecodeSuccessCallback-successCallback-DecodeErrorCallback-errorCallback-DecodeAudioDataOptions-options">
+                decodeAudioData</a>.
+              </dd>
             </dl>
             <p>
               Although the primary method of interfacing with this function is
@@ -1286,9 +1294,13 @@ <h2 id="BaseAudioContext">
               <li>Otherwise:
                 <ol>
                   <li>Take the result, representing the decoded linear PCM
-                  audio data, and resample it to the sample-rate of the
+                  audio data. If <a><code>disableResampling</code></a> is
+                  false, resample it to the sample-rate of the
                   <a><code>AudioContext</code></a> if it is different from the
-                  sample-rate of <a>audioData</a>.
+                  sample-rate of <a>audioData</a>. Otherwise, resampling is
+                  disabled, and the decoded data is unchanged, with the sample
+                  rate of the <a>AudioBuffer</a> set to the sample rate of the
+                  encoded data.
                   </li>
                   <li>Queue a task on the <a>control thread</a>'s event loop to
                   execute the following steps:
@@ -1658,6 +1670,31 @@ <h2 id="BaseAudioContext">
             The error that occurred while decoding.
           </dd>
         </dl>
+        <section>
+          <h2>
+            DecodeAudioDataOptions
+          </h2>
+          <p>
+            This dictionary specifies how <a href=
+            "#widl-BaseAudioContext-decodeAudioData-Promise-AudioBuffer--ArrayBuffer-audioData-DecodeSuccessCallback-successCallback-DecodeErrorCallback-errorCallback-DecodeAudioDataOptions-options">
+            decodeAudioData</a> should process the encoded audio data to
+            produce the decoded linear PCM audio.
+          </p>
+          <dl title="dictionary DecodeAudioDataOptions" class="idl">
+            <dt>
+              boolean disableResampling = false
+            </dt>
+            <dd>
+              Controls whether the decoded audio data returned by <a href=
+              "#widl-BaseAudioContext-decodeAudioData-Promise-AudioBuffer--ArrayBuffer-audioData-DecodeSuccessCallback-successCallback-DecodeErrorCallback-errorCallback-DecodeAudioDataOptions-options">
+              decodeAudioData</a> is automatically resampled to the sample rate
+              of the context. By default, resampling is done. If
+              <a><code>disableResampling</code></a> is true, resampling is not
+              done, and the returned <a>AudioBuffer</a> has its sample rate set
+              to the rate specified by the encoded data.
+            </dd>
+          </dl>
+        </section>
         <section>
           <h3 id="lifetime-AudioContext" class="informative">
             Lifetime