add renderer prop in onBeforeCompile

[sunag]

My need:

To the next NodeMaterial release is need renderer argument to build material. Not to complicate too much the idea is use an auto build since the onBeforeCompile is called.

THREE.NodeMaterial.prototype.onBeforeCompile = function( shader ) {

_this.build( shader.renderer );

};

Renderer argument is the first moment to texture encoding. Now is individual by texture and not by slot as previously, for example:

Old

	if ( builder.isSlot( 'color' ) ) {

		code = 'mapTexelToLinear(' + code + ')';

	} else if ( builder.isSlot( 'emissive' ) ) {

		code = 'emissiveMapTexelToLinear(' + code + ')';

	} else if ( builder.isSlot( 'environment' ) ) {

		code = 'envMapTexelToLinear(' + code + ')';

	}

New

code = builder.material.getTexelDecodingFunction( code, this.value.encoding, builder.renderer );

And fix the fact that the old mode only work with linear texture encoding in NodeMaterial.

[sunag]

Or could be this:

material.onBeforeCompile( materialProperties.shader, _this );

https://github.com/sunag/three.js/blob/a4b8c30be485b4ff661c60151f84ecb57e441227/src/renderers/WebGLRenderer.js#L1541

[mrdoob]

Or could be this:

material.onBeforeCompile( materialProperties.shader, _this );

I think I vote for that instead.

[sunag]

Done!

[WestLangley]

I think there should be an update to the documentation when a method's signature is changed.

[Mugen87]

Current doc: https://rawgit.com/mrdoob/three.js/dev/docs/index.html#api/materials/Material.onBeforeCompile

[sunag]

Is to change it

[method:null onBeforeCompile]()

to?

[method:null onBeforeCompile]( [param:Object shader], [param:WebGLRenderer renderer] )

[mrdoob]

@sunag yep!

[sunag]

Done

[mrdoob]

Thanks!