WebGPURenderer: Remove inline functions in rendering methods.

[PoseidonEnergy]

Related issue: #33107

Description

This PR ensures that no objects are allocated in the Renderer._renderObjectDirect() hot path that gets called for each mesh per render.

From my testing this has produced a 20% gain in the performance of Renderer.render(), plus the added benefit of far less garbage being generated in a render() loop.

You can test this PR live at the following JSFiddle: https://jsfiddle.net/v14csauf

Garbage Accumulation

[github-actions]

📦 Bundle size

Full ESM build, minified and gzipped.

	Before	After	Diff
WebGL	359.27 85.31	359.27 85.31	+0 B +0 B
WebGPU	629.85 174.95	629.94 174.99	+97 B +37 B
WebGPU Nodes	628.43 174.7	628.52 174.74	+97 B +36 B

🌳 Bundle size after tree-shaking

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

	Before	After	Diff
WebGL	491.03 119.72	491.03 119.72	+0 B +0 B
WebGPU	703.47 189.95	703.56 189.99	+97 B +32 B
WebGPU Nodes	652.67 177.34	652.77 177.37	+97 B +31 B

[Mugen87]

Do you mind splitting the PR in smaller changes and just update WebGPURenderer for now?

I think it makes sense to remove the inline functions in draw() as a first step.

[PoseidonEnergy]

Do you mind splitting the PR in smaller changes and just update WebGPURenderer for now?

I think it makes sense to remove the inline functions in draw() as a first step.

Here is the new PR for the BufferGeometry.attributes portion: #33111

For some reason the "Files Changed" comparison view for this PR is all jacked up. All I did was turn the const setPipelineAndBindings = () => { ... } and const draw = () => { ... } functions into private methods on the WebGPUBackend class.

[Mugen87]

Added JSDoc and slightly updated the signature of _draw() in WebGPUBackend.

[Mugen87]

We should avoid object allocation in hot code paths whenever possible so the changes of this PR are indeed wanted.