commit cd4bf9afd97b82279107a966d8ec74406dad5c03
author Jiawei Shao <jiawei.shao@intel.com> Thu Jun 06 01:17:28 2024 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Jun 06 01:17:28 2024 +0000
tree 56095e2ed081c146f509aea324582ae61bd8eabe
parent 57e3d2662faa813f41a8456f142d5fa93fb34ea2

Remove WGSL extension `chromium_internal_dual_source_blending`

This patch removes `chromium_internal_dual_source_blending` from
Dawn after the removal of its last reference in Skia.

This patch also removes the `[dawn]` tag on the blend factors with
`Src1` as the proposal to add dual source blending in WebGPU is
very close to be accepted.

Bug: chromium:341973423
Change-Id: Id661595b3528e1d0a63a62b30efcc94759fe5671
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/191761
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: James Price <jrprice@google.com>
Commit-Queue: Jiawei Shao <jiawei.shao@intel.com>

src/tint/lang/wgsl/extension_test.cc

diff --git a/src/tint/lang/wgsl/extension_test.cc b/src/tint/lang/wgsl/extension_test.cc
index 0237bd5..2e2dba4 100644
--- a/src/tint/lang/wgsl/extension_test.cc
+++ b/src/tint/lang/wgsl/extension_test.cc
@@ -62,7 +62,6 @@
     {"chromium_experimental_pixel_local", Extension::kChromiumExperimentalPixelLocal},
     {"chromium_experimental_push_constant", Extension::kChromiumExperimentalPushConstant},
     {"chromium_experimental_subgroups", Extension::kChromiumExperimentalSubgroups},
-    {"chromium_internal_dual_source_blending", Extension::kChromiumInternalDualSourceBlending},
     {"chromium_internal_graphite", Extension::kChromiumInternalGraphite},
     {"chromium_internal_input_attachments", Extension::kChromiumInternalInputAttachments},
     {"chromium_internal_relaxed_uniform_layout", Extension::kChromiumInternalRelaxedUniformLayout},
@@ -86,24 +85,21 @@
     {"chromiuX_experimentl_sugggroups", Extension::kUndefined},
     {"chromiu_exuerimntal_XVbgroups", Extension::kUndefined},
     {"chromium_experimen3al_subgroups", Extension::kUndefined},
-    {"chromium_internal_dual_soErce_blending", Extension::kUndefined},
-    {"chromiuPP_internal_dual_sourceblenTTing", Extension::kUndefined},
-    {"chromim_internadd_dual_sxxurce_blending", Extension::kUndefined},
-    {"chromi44m_internal_graphite", Extension::kUndefined},
-    {"chromSSuVV_internal_graphite", Extension::kUndefined},
-    {"cRromium_nternR22_graphite", Extension::kUndefined},
-    {"chromium_int9rnaF_inpu_attachments", Extension::kUndefined},
-    {"chrmium_internal_input_attachments", Extension::kUndefined},
-    {"cOOromium_internVlHinput_ttachRRents", Extension::kUndefined},
-    {"chromium_internl_relaxyd_uniform_layout", Extension::kUndefined},
-    {"chromnnum_internrr77_Gelaxell_uniform_layout", Extension::kUndefined},
-    {"chromium_intern4l_relaxe00_uniform_layout", Extension::kUndefined},
-    {"dua_ource_bledoong", Extension::kUndefined},
-    {"dualsorce_blendzzng", Extension::kUndefined},
-    {"ua_sopiirce_bl11nding", Extension::kUndefined},
-    {"f1XX", Extension::kUndefined},
-    {"55199II", Extension::kUndefined},
-    {"frSSHHa", Extension::kUndefined},
+    {"chromium_internal_graphitE", Extension::kUndefined},
+    {"chromium_nternal_gTTPaphite", Extension::kUndefined},
+    {"chxxdomium_interal_graphite", Extension::kUndefined},
+    {"c44romium_internal_input_attachments", Extension::kUndefined},
+    {"chromium_internal_inputSSaVVtachments", Extension::kUndefined},
+    {"chrom22Rm_internal_input_atRchments", Extension::kUndefined},
+    {"chromium_int9rnal_relaxed_Fnifor_layout", Extension::kUndefined},
+    {"chrmium_internal_relaxed_uniform_layout", Extension::kUndefined},
+    {"VRhHomium_internal_relaxd_uniform_OOayout", Extension::kUndefined},
+    {"dual_souyce_bleding", Extension::kUndefined},
+    {"dualrrllnource_blendiG77", Extension::kUndefined},
+    {"dual00source_ble4ding", Extension::kUndefined},
+    {"5", Extension::kUndefined},
+    {"u16", Extension::kUndefined},
+    {"f", Extension::kUndefined},
 };
 
 using ExtensionParseTest = testing::TestWithParam<Case>;