Remove redundant constructors

Bug: chromium:42240266
Change-Id: Ie139029da0e864e595cf996cea8a652709891035
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/211950
Reviewed-by: Corentin Wallez <cwallez@google.com>
Commit-Queue: Jiawei Shao <jiawei.shao@intel.com>
diff --git a/src/dawn/native/PassResourceUsage.cpp b/src/dawn/native/PassResourceUsage.cpp
index 7c1cfa9..977d554 100644
--- a/src/dawn/native/PassResourceUsage.cpp
+++ b/src/dawn/native/PassResourceUsage.cpp
@@ -29,10 +29,6 @@
 
 namespace dawn::native {
 
-ComputePassResourceUsage::ComputePassResourceUsage() = default;
-
-ComputePassResourceUsage::ComputePassResourceUsage(ComputePassResourceUsage&&) = default;
-
 bool operator==(const TextureSyncInfo& a, const TextureSyncInfo& b) {
     return (a.usage == b.usage) && (a.shaderStages == b.shaderStages);
 }
diff --git a/src/dawn/native/PassResourceUsage.h b/src/dawn/native/PassResourceUsage.h
index 2333a20..f85642b 100644
--- a/src/dawn/native/PassResourceUsage.h
+++ b/src/dawn/native/PassResourceUsage.h
@@ -83,12 +83,6 @@
 // textures used, because some unused BindGroups may not be used at all in synchronization
 // scope but their resources still need to be validated on Queue::Submit.
 struct ComputePassResourceUsage {
-    // Somehow without this defaulted constructor, MSVC or its STDlib have an issue where they
-    // use the copy constructor (that's deleted) when doing operations on a
-    // vector<ComputePassResourceUsage>
-    ComputePassResourceUsage(ComputePassResourceUsage&&);
-    ComputePassResourceUsage();
-
     std::vector<SyncScopeResourceUsage> dispatchUsages;
 
     // All the resources referenced by this compute pass for validation in Queue::Submit.