Removed Rotation class. Using getters/setters to keep .rotation and .quaternion in sync.

Author: mrdoob

examples/js/controls/TransformControls.js

@@ -437,9 +437,9 @@ THREE.TransformControls = function ( camera, domElement, doc ) {
 
 						object.rotation.set( 0, 0, 0 );
 
-						if ( name == 'RX' ) object.rotation.setX( Math.atan2( -eye.y, eye.z ) );
-						if ( name == 'RY' ) object.rotation.setY( Math.atan2( eye.x, eye.z ) );
-						if ( name == 'RZ' ) object.rotation.setZ( Math.atan2( eye.y, eye.x ) );
+						if ( name == 'RX' ) object.rotation.x = Math.atan2( -eye.y, eye.z );
+						if ( name == 'RY' ) object.rotation.y = Math.atan2(  eye.x, eye.z );
+						if ( name == 'RZ' ) object.rotation.z = Math.atan2(  eye.y, eye.x );
 
 					}
 

src/core/Object3D.js

@@ -17,11 +17,14 @@ THREE.Object3D = function () {
 	this.up = new THREE.Vector3( 0, 1, 0 );
 
 	this.position = new THREE.Vector3();
+	this.rotation = new THREE.Euler();
 	this.quaternion = new THREE.Quaternion();
 	this.scale = new THREE.Vector3( 1, 1, 1 );
 
-	// for backwards compatibility (maps changes to this.rotation onto this.quaternion)
-	this.rotation = new THREE.Rotation( this.quaternion );
+	// keep rotation and quaternion in sync
+
+	this.rotation._quaternion = this.quaternion;
+	this.quaternion._euler = this.rotation;
 
 	this.renderDepth = null;
 

src/loaders/ObjectLoader.js

@@ -265,8 +265,6 @@ THREE.ObjectLoader.prototype = {
 				matrix.fromArray( data.matrix );
 				matrix.decompose( object.position, object.quaternion, object.scale );
 
-				object.rotation.updateEuler();
-
 			} else {
 
 				if ( data.position !== undefined ) object.position.fromArray( data.position );