Correctly clamp in distance formulas

third_party/WebKit/LayoutTests/webaudio/Panner/panner-distance-clamping.html

Index: third_party/WebKit/LayoutTests/webaudio/Panner/panner-distance-clamping.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/Panner/panner-distance-clamping.html b/third_party/WebKit/LayoutTests/webaudio/Panner/panner-distance-clamping.html
index 78bc17efb3adb2c1d5a5ad92466cf9508f41974c..1fadc29d5f3f15c59f2a45620923b0cfd1cc05a8 100644
--- a/third_party/WebKit/LayoutTests/webaudio/Panner/panner-distance-clamping.html
+++ b/third_party/WebKit/LayoutTests/webaudio/Panner/panner-distance-clamping.html
@@ -195,10 +195,14 @@
         merger.connect(context.destination);
 
         // Move the panner some distance away. Arbitrarily select the x
-        // direction.  For the reference panner, manually clamp the distance to
-        // lie between refDistance and maxDistance.
-        var xRef = Math.min(Math.max(options.distance, pannerRef.refDistance),
-          pannerRef.maxDistance);
+        // direction.  For the reference panner, manually clamp the distance.
+        // All models clamp the distance to a minimum of refDistance.  Only the
+        // linear model also clamps to a maximum of maxDistance.
+        var xRef = Math.max(options.distance, pannerRef.refDistance);;

[hongchan, 2017/05/05 18:07:58]

I see "double semicolon" here and the line 192.

[Raymond Toy, 2017/05/05 18:18:28]

Done.

+
+        if (pannerRef.distanceModel === "linear") {
+          xRef = Math.min(xRef, pannerRef.maxDistance);
+        }
 
         var xTest = options.distance;