VRMLLoader does no error forwarding calling the parser

Description of the problem

When an error occurs while parsing the loaded VRML file, the VRMLLoader does not handle it.

		load: function ( url, onLoad, onProgress, onError ) {

			var scope = this;

			var path = ( scope.path === '' ) ? LoaderUtils.extractUrlBase( url ) : scope.path;

			var loader = new FileLoader( this.manager );
			loader.setPath( scope.path );
			loader.load( url, function ( text ) {

				onLoad( scope.parse( text, path ) );

			}, onProgress, onError );

		},

It should call onError instead.

		load: function ( url, onLoad, onProgress, onError ) {

			var scope = this;

			var path = ( scope.path === '' ) ? LoaderUtils.extractUrlBase( url ) : scope.path;

			var loader = new FileLoader( this.manager );
			loader.setPath( scope.path );
			loader.load( url, function ( text ) {

			  try {
			    const parsed = scope.parse( text, path )
				  onLoad(parsed);

                         } catch (e) {
                            onError(e)
                       }

			}, onProgress, onError );

		},

Three.js version

• r116

Browser

• All of them
• Chrome
• Firefox
• Internet Explorer

OS

• All of them
• Windows
• macOS
• Linux
• Android
• iOS

[Mugen87]

Okay, I’ve added today a PR with the fix.

[mrdoob]

I think loaders should use try/catch. It’s annoying when it bubbles up to the Editor’s Loader.js.