WebGLRenderer: Remove unnecessary __hasExternalTextures check.

[susiwen8]

Description

renderTargetProperties.__hasExternalTextures is set be true just one line above if statement, so I think if can be removed.

Also move some variables to proper place.

[github-actions]

📦 Bundle size

Full ESM build, minified and gzipped.

Filesize dev	Filesize PR	Diff
675.9 kB (167.9 kB)	675.8 kB (167.9 kB)	-27 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

Filesize dev	Filesize PR	Diff
456.4 kB (110.7 kB)	456.4 kB (110.7 kB)	-27 B

[susiwen8]

@Mugen87 I have question about __useRenderToTexture

Looks like __useRenderToTexture would never set be be true. useMultisampledRTT always returns false. Am i missing something?

[Mugen87]

WEBGL_multisampled_render_to_texture is XR related code. I'm not sure about this bit so this should be reviewed from @cabanier.

[cabanier]

WEBGL_multisampled_render_to_texture is XR related code. I'm not sure about this bit so this should be reviewed from @cabanier.

That code is used in other places so it's needed. Specifically, if an experience passes a color and depth buffer and tries to use this extension, we will disable the extension and print this warning.

[cabanier]

Looks like __useRenderToTexture would never set be be true. useMultisampledRTT always returns false. Am i missing something?

That extension is supported by the Quest (and possibly Pico) browser to do efficient multisampled rendering