[Backport] Security bug 378725734 (1/2)

Cherry-pick of patch originally reviewed on
https://dawn-review.googlesource.com/c/dawn/+/214754:
Validate layout constraints for all address spaces

The WGSL spec has a non-normative note that the layout constraints
should be validated for all non-host-shareable address spaces, using
the same constraints as for storage.

See linked bug for details of why this is important.

Bug: 378725734
Change-Id: I3fb02506d8ded000dc3510bdc1b4a24a95089281
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/214754
Reviewed-by: David Neto <[email protected]>
Commit-Queue: James Price <[email protected]>
Reviewed-by: dan sinclair <[email protected]>
Reviewed-on: https://codereview.qt-project.org/c/qt/qtwebengine-chromium/+/609219
Reviewed-by: Anu Aliyas <[email protected]>

Author: jrprice

chromium/third_party/dawn/src/tint/lang/wgsl/resolver/validator.cc

@@ -493,10 +493,6 @@ bool Validator::AddressSpaceLayout(const core::type::Type* store_ty,
         return true;
     }
 
-    if (!core::IsHostShareable(address_space)) {
-        return true;
-    }
-
     auto note_usage = [&] {
         AddNote("'" + store_ty->FriendlyName() + "' used in address space '" +
                     tint::ToString(address_space) + "' here",