Unreviewed, rolling out r110951.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Mar 2012 07:34:51 +0000 (07:34 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Mar 2012 07:34:51 +0000 (07:34 +0000)
http://trac.webkit.org/changeset/110951
https://bugs.webkit.org/show_bug.cgi?id=81316

Does not build on apple-mac (Requested by abarth on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-03-16

Source/WebCore:

* platform/audio/DynamicsCompressor.h:
(DynamicsCompressor):
* webaudio/DynamicsCompressorNode.cpp:
(WebCore::DynamicsCompressorNode::DynamicsCompressorNode):
(WebCore::DynamicsCompressorNode::process):
* webaudio/DynamicsCompressorNode.h:
(DynamicsCompressorNode):
* webaudio/DynamicsCompressorNode.idl:

LayoutTests:

* webaudio/dynamicscompressor-basic-expected.txt:
* webaudio/dynamicscompressor-basic.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@110956 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/webaudio/dynamicscompressor-basic-expected.txt
LayoutTests/webaudio/dynamicscompressor-basic.html
Source/WebCore/ChangeLog
Source/WebCore/platform/audio/DynamicsCompressor.h
Source/WebCore/webaudio/DynamicsCompressorNode.cpp
Source/WebCore/webaudio/DynamicsCompressorNode.h
Source/WebCore/webaudio/DynamicsCompressorNode.idl

index 6b9109d..0ce860e 100644 (file)
@@ -1,3 +1,14 @@
+2012-03-16  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r110951.
+        http://trac.webkit.org/changeset/110951
+        https://bugs.webkit.org/show_bug.cgi?id=81316
+
+        Does not build on apple-mac (Requested by abarth on #webkit).
+
+        * webaudio/dynamicscompressor-basic-expected.txt:
+        * webaudio/dynamicscompressor-basic.html:
+
 2012-03-16  Mike Reed  <reed@google.com>
 
         to be rebaselined after hairline clipping fix in skia lands
index 09c30e6..dd39991 100644 (file)
@@ -5,8 +5,6 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 PASS threshold attribute has correct default value.
 PASS knee attribute has correct default value.
 PASS ratio attribute has correct default value.
-PASS attack attribute has correct default value.
-PASS release attribute has correct default value.
 PASS successfullyParsed is true
 
 TEST COMPLETE
index 87c54d0..d122bec 100644 (file)
@@ -43,16 +43,6 @@ function runTest() {
         else
             testFailed("ratio attribute has incorrect default value.");
         
-        if (compressor.attack.value - 0.003 < 1e-7)
-            testPassed("attack attribute has correct default value.");
-        else
-            testFailed("attack attribute has incorrect default value.");
-
-        if (compressor.release.value - 0.25 < 1e-7)
-            testPassed("release attribute has correct default value.");
-        else
-            testFailed("release attribute has incorrect default value.");
-
     } catch(e) {
         testFailed("Exception thrown when accessing DynamicsCompressorNode attributes.");
     }
index 43c5f78..c43eead 100644 (file)
@@ -1,3 +1,20 @@
+2012-03-16  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r110951.
+        http://trac.webkit.org/changeset/110951
+        https://bugs.webkit.org/show_bug.cgi?id=81316
+
+        Does not build on apple-mac (Requested by abarth on #webkit).
+
+        * platform/audio/DynamicsCompressor.h:
+        (DynamicsCompressor):
+        * webaudio/DynamicsCompressorNode.cpp:
+        (WebCore::DynamicsCompressorNode::DynamicsCompressorNode):
+        (WebCore::DynamicsCompressorNode::process):
+        * webaudio/DynamicsCompressorNode.h:
+        (DynamicsCompressorNode):
+        * webaudio/DynamicsCompressorNode.idl:
+
 2012-03-16  Seo Sanghyeon  <sh4.seo@samsung.com>
 
         Cleanup empty attribute list in IDLs
index f781afe..5201328 100644 (file)
@@ -85,6 +85,8 @@ protected:
     unsigned m_numberOfChannels;
 
     // m_parameters holds the tweakable compressor parameters.
+    // FIXME: expose some of the most important ones (such as threshold, attack, release)
+    // as DynamicsCompressorNode attributes.
     float m_parameters[ParamLast];
     void initializeParameters();
 
index c4e3898..ef70479 100644 (file)
@@ -50,15 +50,11 @@ DynamicsCompressorNode::DynamicsCompressorNode(AudioContext* context, float samp
     m_knee = AudioParam::create("knee", 30, 0, 40);
     m_ratio = AudioParam::create("ratio", 12, 1, 20);
     m_reduction = AudioParam::create("reduction", 0, -20, 0);
-    m_attack = AudioParam::create("attack", 0.003, 0, 1);
-    m_release = AudioParam::create("release", 0.250, 0, 1);
 
     m_threshold->setContext(context);
     m_knee->setContext(context);
     m_ratio->setContext(context);
     m_reduction->setContext(context);
-    m_attack->setContext(context);
-    m_release->setContext(context);
 
     initialize();
 }
@@ -76,14 +72,10 @@ void DynamicsCompressorNode::process(size_t framesToProcess)
     float threshold = m_threshold->value();
     float knee = m_knee->value();
     float ratio = m_ratio->value();
-    float attack = m_attack->value();
-    float release = m_release->value();
 
     m_dynamicsCompressor->setParameterValue(DynamicsCompressor::ParamThreshold, threshold);
     m_dynamicsCompressor->setParameterValue(DynamicsCompressor::ParamKnee, knee);
     m_dynamicsCompressor->setParameterValue(DynamicsCompressor::ParamRatio, ratio);
-    m_dynamicsCompressor->setParameterValue(DynamicsCompressor::ParamAttack, attack);
-    m_dynamicsCompressor->setParameterValue(DynamicsCompressor::ParamRelease, release);
 
     m_dynamicsCompressor->process(input(0)->bus(), outputBus, framesToProcess);
 
index 338dff3..cf7e63a 100644 (file)
@@ -52,8 +52,6 @@ public:
     AudioParam* threshold() { return m_threshold.get(); }
     AudioParam* knee() { return m_knee.get(); }
     AudioParam* ratio() { return m_ratio.get(); }
-    AudioParam* attack() { return m_attack.get(); }
-    AudioParam* release() { return m_release.get(); }
 
     // Amount by which the compressor is currently compressing the signal in decibels.
     AudioParam* reduction() { return m_reduction.get(); }
@@ -69,8 +67,6 @@ private:
     RefPtr<AudioParam> m_knee;
     RefPtr<AudioParam> m_ratio;
     RefPtr<AudioParam> m_reduction;
-    RefPtr<AudioParam> m_attack;
-    RefPtr<AudioParam> m_release;
 };
 
 } // namespace WebCore
index a2e3565..60d64fd 100644 (file)
@@ -31,7 +31,5 @@ module audio {
         readonly attribute AudioParam knee; // in Decibels
         readonly attribute AudioParam ratio; // unit-less
         readonly attribute AudioParam reduction; // in Decibels
-        readonly attribute AudioParam attack; // in Seconds
-        readonly attribute AudioParam release; // in Seconds
     };
 }