Revert "Bug 1966669: Complete remaining WebGLTexelFormat conversions. r=ahale" for causing webgpu crashes mozilla::ConvertImage

marco-c · marco-c · commit 113f632192e9 · 2025-10-08T14:03:08.000Z
This reverts commit 4952555f14c530954273bb9f038b347ec56e5266.

UltraBlame original commit: 9af212a90b499d56ea4c1664809d159266aba274
diff --git a/dom/canvas/WebGLTexelConversions.cpp b/dom/canvas/WebGLTexelConversions.cpp
@@ -2276,14 +2276,14 @@ WEBGLIMAGECONVERTER_CASE_DSTFORMAT
 WebGLTexelFormat
 :
 :
-RGB8
+RGB565
 )
 WEBGLIMAGECONVERTER_CASE_DSTFORMAT
 (
 WebGLTexelFormat
 :
 :
-RGB565
+RGB8
 )
 WEBGLIMAGECONVERTER_CASE_DSTFORMAT
 (
@@ -2317,7 +2317,7 @@ WEBGLIMAGECONVERTER_CASE_DSTFORMAT
 WebGLTexelFormat
 :
 :
-RGBA8
+RGBA4444
 )
 WEBGLIMAGECONVERTER_CASE_DSTFORMAT
 (
@@ -2331,7 +2331,7 @@ WEBGLIMAGECONVERTER_CASE_DSTFORMAT
 WebGLTexelFormat
 :
 :
-RGBA4444
+RGBA8
 )
 WEBGLIMAGECONVERTER_CASE_DSTFORMAT
 (
@@ -2347,6 +2347,13 @@ WebGLTexelFormat
 :
 RGBA32F
 )
+WEBGLIMAGECONVERTER_CASE_DSTFORMAT
+(
+WebGLTexelFormat
+:
+:
+BGRA8
+)
 default
 :
 MOZ_ASSERT
@@ -2494,27 +2501,6 @@ WebGLTexelFormat
 :
 RA32F
 )
-WEBGLIMAGECONVERTER_CASE_SRCFORMAT
-(
-WebGLTexelFormat
-:
-:
-RG8
-)
-WEBGLIMAGECONVERTER_CASE_SRCFORMAT
-(
-WebGLTexelFormat
-:
-:
-RG16F
-)
-WEBGLIMAGECONVERTER_CASE_SRCFORMAT
-(
-WebGLTexelFormat
-:
-:
-RG32F
-)
 /
 /
 3
@@ -2526,21 +2512,14 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 WebGLTexelFormat
 :
 :
-RGB8
-)
-WEBGLIMAGECONVERTER_CASE_SRCFORMAT
-(
-WebGLTexelFormat
-:
-:
 RGB565
 )
 WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 (
 WebGLTexelFormat
 :
 :
-RGB11F11F10F
+RGB8
 )
 WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 (
@@ -2567,7 +2546,7 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 WebGLTexelFormat
 :
 :
-RGBA8
+RGBA4444
 )
 WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 (
@@ -2581,7 +2560,7 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 WebGLTexelFormat
 :
 :
-RGBA4444
+RGBA8
 )
 WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 (
@@ -2608,13 +2587,6 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT
 WebGLTexelFormat
 :
 :
-RGBX8
-)
-WEBGLIMAGECONVERTER_CASE_SRCFORMAT
-(
-WebGLTexelFormat
-:
-:
 BGRX8
 )
 WEBGLIMAGECONVERTER_CASE_SRCFORMAT

Original file line number	Diff line number	Diff line change
`@@ -2276,14 +2276,14 @@ WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
`2276`	`2276`	`WebGLTexelFormat`
`2277`	`2277`	`:`
`2278`	`2278`	`:`
`2279`		`-RGB8`
	`2279`	`+RGB565`
`2280`	`2280`	`)`
`2281`	`2281`	`WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
`2282`	`2282`	`(`
`2283`	`2283`	`WebGLTexelFormat`
`2284`	`2284`	`:`
`2285`	`2285`	`:`
`2286`		`-RGB565`
	`2286`	`+RGB8`
`2287`	`2287`	`)`
`2288`	`2288`	`WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
`2289`	`2289`	`(`
`@@ -2317,7 +2317,7 @@ WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
`2317`	`2317`	`WebGLTexelFormat`
`2318`	`2318`	`:`
`2319`	`2319`	`:`
`2320`		`-RGBA8`
	`2320`	`+RGBA4444`
`2321`	`2321`	`)`
`2322`	`2322`	`WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
`2323`	`2323`	`(`
`@@ -2331,7 +2331,7 @@ WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
`2331`	`2331`	`WebGLTexelFormat`
`2332`	`2332`	`:`
`2333`	`2333`	`:`
`2334`		`-RGBA4444`
	`2334`	`+RGBA8`
`2335`	`2335`	`)`
`2336`	`2336`	`WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
`2337`	`2337`	`(`
`@@ -2347,6 +2347,13 @@ WebGLTexelFormat`
`2347`	`2347`	`:`
`2348`	`2348`	`RGBA32F`
`2349`	`2349`	`)`
	`2350`	`+WEBGLIMAGECONVERTER_CASE_DSTFORMAT`
	`2351`	`+(`
	`2352`	`+WebGLTexelFormat`
	`2353`	`+:`
	`2354`	`+:`
	`2355`	`+BGRA8`
	`2356`	`+)`
`2350`	`2357`	`default`
`2351`	`2358`	`:`
`2352`	`2359`	`MOZ_ASSERT`
`@@ -2494,27 +2501,6 @@ WebGLTexelFormat`
`2494`	`2501`	`:`
`2495`	`2502`	`RA32F`
`2496`	`2503`	`)`
`2497`		`-WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2498`		`-(`
`2499`		`-WebGLTexelFormat`
`2500`		`-:`
`2501`		`-:`
`2502`		`-RG8`
`2503`		`-)`
`2504`		`-WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2505`		`-(`
`2506`		`-WebGLTexelFormat`
`2507`		`-:`
`2508`		`-:`
`2509`		`-RG16F`
`2510`		`-)`
`2511`		`-WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2512`		`-(`
`2513`		`-WebGLTexelFormat`
`2514`		`-:`
`2515`		`-:`
`2516`		`-RG32F`
`2517`		`-)`
`2518`	`2504`	`/`
`2519`	`2505`	`/`
`2520`	`2506`	`3`
`@@ -2526,21 +2512,14 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2526`	`2512`	`WebGLTexelFormat`
`2527`	`2513`	`:`
`2528`	`2514`	`:`
`2529`		`-RGB8`
`2530`		`-)`
`2531`		`-WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2532`		`-(`
`2533`		`-WebGLTexelFormat`
`2534`		`-:`
`2535`		`-:`
`2536`	`2515`	`RGB565`
`2537`	`2516`	`)`
`2538`	`2517`	`WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2539`	`2518`	`(`
`2540`	`2519`	`WebGLTexelFormat`
`2541`	`2520`	`:`
`2542`	`2521`	`:`
`2543`		`-RGB11F11F10F`
	`2522`	`+RGB8`
`2544`	`2523`	`)`
`2545`	`2524`	`WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2546`	`2525`	`(`
`@@ -2567,7 +2546,7 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2567`	`2546`	`WebGLTexelFormat`
`2568`	`2547`	`:`
`2569`	`2548`	`:`
`2570`		`-RGBA8`
	`2549`	`+RGBA4444`
`2571`	`2550`	`)`
`2572`	`2551`	`WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2573`	`2552`	`(`
`@@ -2581,7 +2560,7 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2581`	`2560`	`WebGLTexelFormat`
`2582`	`2561`	`:`
`2583`	`2562`	`:`
`2584`		`-RGBA4444`
	`2563`	`+RGBA8`
`2585`	`2564`	`)`
`2586`	`2565`	`WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2587`	`2566`	`(`
`@@ -2608,13 +2587,6 @@ WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2608`	`2587`	`WebGLTexelFormat`
`2609`	`2588`	`:`
`2610`	`2589`	`:`
`2611`		`-RGBX8`
`2612`		`-)`
`2613`		`-WEBGLIMAGECONVERTER_CASE_SRCFORMAT`
`2614`		`-(`
`2615`		`-WebGLTexelFormat`
`2616`		`-:`
`2617`		`-:`
`2618`	`2590`	`BGRX8`
`2619`	`2591`	`)`
`2620`	`2592`	`WEBGLIMAGECONVERTER_CASE_SRCFORMAT`