commit fa4b74d479aa9cf41e4aa067817bad47e97f3b63
author Ryan Harrison <rharrison@chromium.org> Wed Aug 05 18:18:04 2020 +0000
committer Commit Bot service account <commit-bot@chromium.org> Wed Aug 05 18:18:04 2020 +0000
tree ab093372f13a78d50696b2233f7187734f1a36b2
parent 7ce4924a35eac58c0c44bd1b561b76d094ff6dee

Remove temporary disabling of validation check

Change-Id: I8bf7f4850689c240152a7a87ed42d7637c69ed45
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/26380
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>

src/dawn_native/ShaderModule.cpp

diff --git a/src/dawn_native/ShaderModule.cpp b/src/dawn_native/ShaderModule.cpp
index 402a47f..fbbe114 100644
--- a/src/dawn_native/ShaderModule.cpp
+++ b/src/dawn_native/ShaderModule.cpp
@@ -356,13 +356,11 @@
             return DAWN_VALIDATION_ERROR(errorStream.str().c_str());
         }
 
-        // TODO(rharrsion): Add this back in once multual dependencies are
-        // rolled into Chromium.
-        // tint::Validator validator;
-        // if (!validator.Validate(&module)) {
-        //    errorStream << "Validation: " << validator.error() << std::endl;
-        //    return DAWN_VALIDATION_ERROR(errorStream.str().c_str());
-        // }
+        tint::Validator validator;
+        if (!validator.Validate(&module)) {
+            errorStream << "Validation: " << validator.error() << std::endl;
+            return DAWN_VALIDATION_ERROR(errorStream.str().c_str());
+        }
 
         return {};
     }