Mirrors/Tone.js
2
0
Fork 0
mirror of https://github.com/Tonejs/Tone.js synced 2024-11-16 16:48:00 +00:00
Code Issues Projects Releases Packages Wiki Activity

removing unnecessary safari inconsistency bug

Browse source
This commit is contained in:
Yotam Mann 2015-08-24 17:30:11 -04:00
parent 499bb57ac6
commit 102262cea9
1 changed files with 5 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
Tone/signal/WaveShaper.js
View file

@ -90,11 +90,6 @@ define(["Tone/core/Tone", "Tone/signal/SignalBase"], function(Tone){
return this._shaper.curve;
},
set : function(mapping){
//fixes safari WaveShaperNode bug
if (this._isSafari()){
var first = mapping[0];
mapping.unshift(first);
}
this._curve = new Float32Array(mapping);
this._shaper.curve = this._curve;
}
@ -112,20 +107,14 @@ define(["Tone/core/Tone", "Tone/signal/SignalBase"], function(Tone){
return this._shaper.oversample;
},
set : function(oversampling){
this._shaper.oversample = oversampling;
if (["none", "2x", "4x"].indexOf(oversampling) !== -1){
this._shaper.oversample = oversampling;
} else {
throw new Error("invalid oversampling: "+oversampling);
}
}
});
/**
* returns true if the browser is safari
* @return {boolean}
* @private
*/
Tone.WaveShaper.prototype._isSafari = function(){
var ua = navigator.userAgent.toLowerCase();
return ua.indexOf("safari") !== -1 && ua.indexOf("chrome") === -1;
};
/**
* Clean up.
* @returns {Tone.WaveShaper} this