Change ImageUtils.getDataURL() to accept a non-renderable object and return an empty string if a data URL cannot be generated. Allows the serialization of a scene with a DataTexture/WebGLRenderTarget after mrdoob#16651 landed without throwing. Fixes mrdoob#16763.

Author: jsantell

src/extras/ImageUtils.d.ts

@@ -7,6 +7,12 @@ import { Texture } from '../textures/Texture';
  * @deprecated Use {@link TextureLoader} instead.
  */
 export namespace ImageUtils {
+	/**
+	 */
+	export function getDataURL(
+		image: any,
+	): string;
+
 	/**
 	 * @deprecated
 	 */

src/extras/ImageUtils.js

@@ -12,7 +12,11 @@ var ImageUtils = {
 
 		var canvas;
 
-		if ( typeof HTMLCanvasElement == 'undefined' ) {
+		if ( ! image ) {
+
+			return '';
+
+		} else if ( typeof HTMLCanvasElement == 'undefined' ) {
 
 			return image.src;
 
@@ -35,7 +39,18 @@ var ImageUtils = {
 
 			} else {
 
-				context.drawImage( image, 0, 0, image.width, image.height );
+				try {
+
+					// Can throw if `image` is not a drawable type, e.g. HTMLImageElement, HTMLVideoElement,
+					// ImageBitmap; occurs for DataTextures, Textures from WebGLRenderTarget etc..
+
+					context.drawImage( image, 0, 0, image.width, image.height );
+
+				} catch ( e ) {
+
+					return '';
+
+				}
 
 			}