Remove unused "ValidateVideoTextureCanBeShared"
ValidateVideoTextureCanBeShared() is unused in current dawn.
According to discussion in https://dawn-review.googlesource.com/c/dawn/+/174722:
"Import of multiplanar DXGI HANDLE also does not need a feature exposed because assuming the caller passes a valid HANDLE, then the caller needs to have successfully created that HANDLE. That means the caller needs to have successfully exported that HANDLE - which would not be possible if the device did not have sharedResourceTier2. In short, Dawn doesn't need the feature for import operations. We would need to expose such a feature if we allowed export of multiplanar share handles, but we don't have export today."
Bug: 351953347
Change-Id: Ie41be1e99587d298b4361a2d9d5bfab00c52ca07
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/197414
Commit-Queue: Shaobo Yan <shaobo.yan@intel.com>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
diff --git a/src/dawn/native/d3d11/TextureD3D11.cpp b/src/dawn/native/d3d11/TextureD3D11.cpp
index f8f9bda..f0f6776 100644
--- a/src/dawn/native/d3d11/TextureD3D11.cpp
+++ b/src/dawn/native/d3d11/TextureD3D11.cpp
@@ -146,23 +146,6 @@
} // namespace
-MaybeError ValidateVideoTextureCanBeShared(Device* device, DXGI_FORMAT textureFormat) {
- const bool supportsSharedResourceCapabilityTier2 =
- device->GetDeviceInfo().supportsSharedResourceCapabilityTier2;
- switch (textureFormat) {
- case DXGI_FORMAT_NV12:
- case DXGI_FORMAT_P010:
- if (supportsSharedResourceCapabilityTier2) {
- return {};
- }
- break;
- default:
- break;
- }
-
- return DAWN_VALIDATION_ERROR("DXGI format does not support cross-API sharing.");
-}
-
// static
ResultOrError<Ref<Texture>> Texture::Create(Device* device,
const UnpackedPtr<TextureDescriptor>& descriptor) {
diff --git a/src/dawn/native/d3d11/TextureD3D11.h b/src/dawn/native/d3d11/TextureD3D11.h
index b4c4741..b008fe0 100644
--- a/src/dawn/native/d3d11/TextureD3D11.h
+++ b/src/dawn/native/d3d11/TextureD3D11.h
@@ -53,8 +53,6 @@
class ScopedCommandRecordingContext;
class SharedTextureMemory;
-MaybeError ValidateVideoTextureCanBeShared(Device* device, DXGI_FORMAT textureFormat);
-
class Texture final : public d3d::Texture {
public:
static ResultOrError<Ref<Texture>> Create(Device* device,
diff --git a/src/dawn/native/d3d12/TextureD3D12.cpp b/src/dawn/native/d3d12/TextureD3D12.cpp
index d8635b5..0c2b47a 100644
--- a/src/dawn/native/d3d12/TextureD3D12.cpp
+++ b/src/dawn/native/d3d12/TextureD3D12.cpp
@@ -137,24 +137,6 @@
} // namespace
-// https://docs.microsoft.com/en-us/windows/win32/api/d3d12/ne-d3d12-d3d12_shared_resource_compatibility_tier
-MaybeError ValidateVideoTextureCanBeShared(Device* device, DXGI_FORMAT textureFormat) {
- const bool supportsSharedResourceCapabilityTier1 =
- device->GetDeviceInfo().supportsSharedResourceCapabilityTier1;
- switch (textureFormat) {
- // MSDN docs are not correct, NV12 requires at-least tier 1.
- case DXGI_FORMAT_NV12:
- if (supportsSharedResourceCapabilityTier1) {
- return {};
- }
- break;
- default:
- break;
- }
-
- return DAWN_VALIDATION_ERROR("DXGI format does not support cross-API sharing.");
-}
-
// static
ResultOrError<Ref<Texture>> Texture::Create(Device* device,
const UnpackedPtr<TextureDescriptor>& descriptor) {
diff --git a/src/dawn/native/d3d12/TextureD3D12.h b/src/dawn/native/d3d12/TextureD3D12.h
index 3c8a943..02388d9 100644
--- a/src/dawn/native/d3d12/TextureD3D12.h
+++ b/src/dawn/native/d3d12/TextureD3D12.h
@@ -51,8 +51,6 @@
class CommandRecordingContext;
class Device;
-MaybeError ValidateVideoTextureCanBeShared(Device* device, DXGI_FORMAT textureFormat);
-
class Texture final : public d3d::Texture {
public:
static ResultOrError<Ref<Texture>> Create(Device* device,
