Removed yet some more .translate() and .rotateByAxis() instances. See #3185.

src/extras/helpers/SpotLightHelper.js

@@ -18,7 +18,8 @@ THREE.SpotLightHelper = function ( light, sphereSize ) {
 	this.add( this.lightSphere );
 
 	geometry = new THREE.CylinderGeometry( 0.0001, 1, 1, 8, 1, true );
-	geometry.applyMatrix( new THREE.Matrix4().makeRotationX( - Math.PI / 2 ).translate( new THREE.Vector3( 0, -0.5, 0 ) ) );
+	geometry.applyMatrix( new THREE.Matrix4().makeTranslation( 0, -0.5, 0 ) );
+	geometry.applyMatrix( new THREE.Matrix4().makeRotationX( - Math.PI / 2 ) );
 
 	material = new THREE.MeshBasicMaterial( { fog: false, wireframe: true, opacity: 0.3, transparent: true } );
 	material.color.copy( this.light.color ).multiplyScalar( this.light.intensity );

test/unit/math/Ray.js

@@ -151,15 +151,15 @@ test( "intersectPlane", function() {
 
 test( "applyMatrix4", function() {
 	var a = new THREE.Ray( one3.clone(), new THREE.Vector3( 0, 0, 1 ) );
-	var m = new THREE.Matrix4().identity();
+	var m = new THREE.Matrix4();
 
 	ok( a.clone().applyMatrix4( m ).equals( a ), "Passed!" );
 
 	a = new THREE.Ray( zero3.clone(), new THREE.Vector3( 0, 0, 1 ) );
-	m.rotateByAxis( new THREE.Vector3( 0, 0, 1 ), Math.PI );
+	m.makeRotationZ( Math.PI );
 	ok( a.clone().applyMatrix4( m ).equals( a ), "Passed!" );
 
-	m.identity().rotateX( Math.PI );
+	m.makeRotationX( Math.PI );
 	var b = a.clone();
 	b.direction.negate();
 	var a2 = a.clone().applyMatrix4( m );