setFaceCulling not working for the renderer

[boyravikumar]

(*** This section is for bug reports and feature requests only. This is NOT a help site. Do not ask help questions here. If you need help, please use stackoverflow. ***)

Description of the problem

Three.js version

• Dev
• r82
• ...

Browser

• All of them
• Chrome
• Firefox
• Internet Explorer

OS

• All of them
• Windows
• Linux
• Android
• IOS

Hardware Requirements (graphics card, VR Device, ...)

[takahirox]

Maybe because material.side overrides cull_face setting in rendering.

three.js/src/renderers/WebGLRenderer.js

Lines 1645 to 1663 in 6404d18

	function setMaterial( material ) {
	material.side === DoubleSide
	? state.disable( _gl.CULL_FACE )
	: state.enable( _gl.CULL_FACE );
	state.setFlipSided( material.side === BackSide );
	material.transparent === true
	? state.setBlending( material.blending, material.blendEquation, material.blendSrc, material.blendDst, material.blendEquationAlpha, material.blendSrcAlpha, material.blendDstAlpha, material.premultipliedAlpha )
	: state.setBlending( NoBlending );
	state.setDepthFunc( material.depthFunc );
	state.setDepthTest( material.depthTest );
	state.setDepthWrite( material.depthWrite );
	state.setColorWrite( material.colorWrite );
	state.setPolygonOffset( material.polygonOffset, material.polygonOffsetFactor, material.polygonOffsetUnits );
	}

I'm wondering what WebGLRenderer.setFaceCulling() is for...

[Mugen87]

I would say we can remove WebGLRenderer.setFaceCulling().

@mrdoob @WestLangley What do you think about this issue?

[WestLangley]

It is OK to remove it. Users can directly call the methods it calls -- if they dare to do so...