Animation: Remove automatic track validate/optimize.

docs/api/animation/AnimationClip.html

@@ -69,7 +69,7 @@
 		<h3>[method:this optimize]()</h3>
 		<p>
 			Optimizes each track by removing equivalent sequential keys (which are common in morph target
-			sequences). Called automatically by [page:AnimationMixer.clipAction mixer.clipAction]().
+			sequences).
 		</p>
 
 		<h3>[method:this resetDuration]()</h3>
@@ -85,8 +85,7 @@
 
 		<h3>[method:Boolean validate]()</h3>
 		<p>
-			Performs minimal validation on each track in the clip. Called automatically by
-			[page:AnimationMixer.clipAction mixer.clipAction](). Returns true if all tracks are valid.
+			Performs minimal validation on each track in the clip. Returns true if all tracks are valid.
 		</p>
 
 

docs/api/animation/AnimationMixer.html

@@ -49,15 +49,12 @@
 		<h2>Methods</h2>
 
 
-		<h3>[method:AnimationAction clipAction]([param:AnimationClip clip], [param:Object3D optionalRoot], [param:Boolean needsValidateAndOptimize])</h3>
+		<h3>[method:AnimationAction clipAction]([param:AnimationClip clip], [param:Object3D optionalRoot])</h3>
 		<p>
 			Returns an [page:AnimationAction] for the passed clip, optionally using a root object different
 			from the mixer's default root. The first parameter can be either an [page:AnimationClip] object
 			or the name of an AnimationClip.<br /><br />
 
-			Automatically calls .validate() and .optimize() on the clip, unless 'needsValidateAndOptimize'
-			parameter is set to false.<br /><br />
-
 			If an action fitting the clip and root parameters doesn't yet exist, it will be created by
 			this method. Calling this method several times with the same clip and root parameters always
 			returns the same clip instance.

docs/api/animation/KeyframeTrack.html

@@ -205,7 +205,6 @@
 		<h3>[method:this optimize]()</h3>
 		<p>
 			Removes equivalent sequential keys, which are common in morph target sequences.
-			Called automatically by [page:AnimationMixer.clipAction mixer.clipAction]().
 		</p>
 
 		<h3>[method:this scale]()</h3>
@@ -236,8 +235,7 @@
 
 		<h3>[method:Boolean validate]()</h3>
 		<p>
-			Performs minimal validation on the tracks. Called automatically by
-			[page:AnimationMixer.clipAction mixer.clipAction](). Returns true if valid.
+			Performs minimal validation on the tracks. Returns true if valid.
 		</p>
 
 		<p>

src/animation/AnimationMixer.js

@@ -518,7 +518,7 @@ AnimationMixer.prototype = Object.assign( Object.create( EventDispatcher.prototy
 	// return an action for a clip optionally using a custom root target
 	// object (this method allocates a lot of dynamic memory in case a
 	// previously unknown clip/root combination is specified)
-	clipAction: function ( clip, optionalRoot, needsValidateAndOptimize ) {
+	clipAction: function ( clip, optionalRoot ) {
 
 		var root = optionalRoot || this._root,
 			rootUuid = root.uuid,
@@ -555,12 +555,6 @@ AnimationMixer.prototype = Object.assign( Object.create( EventDispatcher.prototy
 		// clip must be known when specified via string
 		if ( clipObject === null ) return null;
 
-		if ( needsValidateAndOptimize !== false ) {
-
-			if ( clipObject.validate() ) clipObject.optimize();
-
-		}
-
 		// allocate all resources required to run it
 		var newAction = new AnimationAction( this, clipObject, optionalRoot );
 

src/animation/KeyframeTrack.js

@@ -34,12 +34,6 @@ function KeyframeTrack( name, times, values, interpolation ) {
 	this.times = AnimationUtils.convertArray( times, this.TimeBufferType );
 	this.values = AnimationUtils.convertArray( values, this.ValueBufferType );
 
-	// TODO: These flags are used to warn users of duplicate validate/optimize
-	// calls when multiple actions are created from a single clip, starting with
-	// r95. We will eventually remove these flags and the transitional warnings.
-	this._needsValidate = true;
-	this._needsOptimize = true;
-
 	this.setInterpolation( interpolation || this.DefaultInterpolation );
 
 }
@@ -358,14 +352,6 @@ Object.assign( KeyframeTrack.prototype, {
 	// ensure we do not get a GarbageInGarbageOut situation, make sure tracks are at least minimally viable
 	validate: function () {
 
-		if ( ! this._needsValidate ) {
-
-			console.warn( 'THREE.KeyframeTrack: Track has already been validated. '
-				+ 'Disable "needsValidateAndOptimize" argument to mixer.clipAction() '
-				+ 'to avoid redundant validation.' );
-
-		}
-
 		var valid = true;
 
 		var valueSize = this.getValueSize();
@@ -436,8 +422,6 @@ Object.assign( KeyframeTrack.prototype, {
 
 		}
 
-		this._needsValidate = !valid;
-
 		return valid;
 
 	},
@@ -446,14 +430,6 @@ Object.assign( KeyframeTrack.prototype, {
 	// (0,0,0,0,1,1,1,0,0,0,0,0,0,0) --> (0,0,1,1,0,0)
 	optimize: function () {
 
-		if ( ! this._needsOptimize ) {
-
-			console.warn( 'THREE.KeyframeTrack: Track has already been optimized. '
-				+ 'Disable "needsValidateAndOptimize" argument to mixer.clipAction() '
-				+ 'to avoid redundant optimization.' );
-
-		}
-
 		var times = this.times,
 			values = this.values,
 			stride = this.getValueSize(),
@@ -552,8 +528,6 @@ Object.assign( KeyframeTrack.prototype, {
 
 		}
 
-		this._needsOptimize = false;
-
 		return this;
 
 	}