commit 441af7d8ab5f5d8387ecdfc28143e88c40a0bc98
author Shrek Shao <shrekshao@google.com> Tue Dec 10 21:09:26 2024 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Dec 10 21:09:26 2024 +0000
tree bc61a426b2a3aa7f1e3842d6f03f5765c1143d9d
parent 0602fb000982f1beb6eac3d705e13c864e785732

[Compat] Remove GL requiring color_buffer_(half_)float extensions

They are optional feature now:
https://github.com/gpuweb/gpuweb/pull/4983

Yet compat feature level is unsettled. Remove to unblock
requesting a GL context without extensions.

Bug: 381214487, 382084196
Change-Id: I4c57defbebd4d9d68abe2b0b20eb6c50cafde953
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/218700
Commit-Queue: Shrek Shao <shrekshao@google.com>
Reviewed-by: Stephen White <senorblanco@chromium.org>
Reviewed-by: Gregg Tavares <gman@chromium.org>

src/dawn/native/opengl/PhysicalDeviceGL.cpp

diff --git a/src/dawn/native/opengl/PhysicalDeviceGL.cpp b/src/dawn/native/opengl/PhysicalDeviceGL.cpp
index 2793016..f9df651 100644
--- a/src/dawn/native/opengl/PhysicalDeviceGL.cpp
+++ b/src/dawn/native/opengl/PhysicalDeviceGL.cpp
@@ -161,15 +161,6 @@
 
     if (mFunctions.GetVersion().IsES()) {
         DAWN_ASSERT(GetBackendType() == wgpu::BackendType::OpenGLES);
-
-        // WebGPU requires being able to render to f16 and being able to blend f16
-        // which EXT_color_buffer_half_float provides.
-        DAWN_INVALID_IF(!mFunctions.IsGLExtensionSupported("GL_EXT_color_buffer_half_float"),
-                        "GL_EXT_color_buffer_half_float is required");
-
-        // WebGPU requires being able to render to f32 but does not require being able to blend f32
-        DAWN_INVALID_IF(!mFunctions.IsGLExtensionSupported("GL_EXT_color_buffer_float"),
-                        "GL_EXT_color_buffer_float is required");
     } else {
         DAWN_ASSERT(GetBackendType() == wgpu::BackendType::OpenGL);
     }