commit cbe795822ae42b4b8751f377fe6e9044ad12dd7e
author Jiawei Shao <jiawei.shao@intel.com> Mon May 13 04:08:15 2024 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon May 13 04:08:15 2024 +0000
tree 79c54a3b6bb38601a824cbdf1fc1a24ca775545c
parent 566aef68d197ee1c8ee366364f9059b7cc807eff

Remove deprecated `APIReference()`

This patch removes the deprecated API `APIReference()` from Dawn.

Bug: dawn:2234
Change-Id: I238f65c5e49d0abd04ee9c66bb0c0c52e73822ea
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/187707
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Jiawei Shao <jiawei.shao@intel.com>

generator/dawn_json_generator.py

diff --git a/generator/dawn_json_generator.py b/generator/dawn_json_generator.py
index 8da2fb0..a5b9b7e 100644
--- a/generator/dawn_json_generator.py
+++ b/generator/dawn_json_generator.py
@@ -964,8 +964,6 @@
 
 def c_methods(params, typ):
     return typ.methods + [
-        # TODO(dawn:2234): Deprecated. Remove when no longer used.
-        Method(Name('reference'), params['types']['void'], [], False, {}),
         Method(Name('add ref'), params['types']['void'], [], False, {}),
         Method(Name('release'), params['types']['void'], [], False, {}),
     ]

generator/templates/api.h

diff --git a/generator/templates/api.h b/generator/templates/api.h
index 43b495e..80d4991 100644
--- a/generator/templates/api.h
+++ b/generator/templates/api.h
@@ -73,8 +73,6 @@
 #include <stdint.h>
 #include <stddef.h>
 
-#define WGPU_BREAKING_REFERENCE_ADDREF
-
 #if defined(__cplusplus)
 #  if __cplusplus >= 201103L
 #    define {{API}}_MAKE_INIT_STRUCT(type, value) (type value)

src/dawn/common/RefCounted.h

diff --git a/src/dawn/common/RefCounted.h b/src/dawn/common/RefCounted.h
index 0d06e04..cf062d2 100644
--- a/src/dawn/common/RefCounted.h
+++ b/src/dawn/common/RefCounted.h
@@ -76,8 +76,6 @@
     // synchronization in place for destruction.
     void Release();
 
-    // TODO(dawn:2234): Deprecated. Remove when no longer used.
-    void APIReference() { APIAddRef(); }
     void APIAddRef() { AddRef(); }
     void APIRelease() { Release(); }
 

src/dawn/fuzzers/DawnWireServerFuzzer.cpp

diff --git a/src/dawn/fuzzers/DawnWireServerFuzzer.cpp b/src/dawn/fuzzers/DawnWireServerFuzzer.cpp
index 3797267..5512b5d 100644
--- a/src/dawn/fuzzers/DawnWireServerFuzzer.cpp
+++ b/src/dawn/fuzzers/DawnWireServerFuzzer.cpp
@@ -120,7 +120,7 @@
         for (dawn::native::Adapter adapter : adapters) {
             if (sAdapterSupported(adapter)) {
                 WGPUAdapter cAdapter = adapter.Get();
-                dawn::native::GetProcs().adapterReference(cAdapter);
+                dawn::native::GetProcs().adapterAddRef(cAdapter);
                 callback(WGPURequestAdapterStatus_Success, cAdapter, nullptr, userdata);
                 return;
             }

src/dawn/native/RefCountedWithExternalCount.h

diff --git a/src/dawn/native/RefCountedWithExternalCount.h b/src/dawn/native/RefCountedWithExternalCount.h
index 64b4e5d..524e988 100644
--- a/src/dawn/native/RefCountedWithExternalCount.h
+++ b/src/dawn/native/RefCountedWithExternalCount.h
@@ -48,8 +48,6 @@
     using T::Release;
     using T::T;
 
-    // TODO(dawn:2234): Deprecated. Remove when no longer used.
-    void APIReference() { APIAddRef(); }
     void APIAddRef() {
         IncrementExternalRefCount();
         T::APIAddRef();
@@ -59,7 +57,7 @@
         if (mExternalRefCount.Decrement()) {
             WillDropLastExternalRef();
         }
-        T::APIRelease();
+        T::Release();
     }
 
     void IncrementExternalRefCount() { mExternalRefCount.Increment(); }