commit a32e3bd01402fd9a9aa7a626295162840d8fde50
author Corentin Wallez <cwallez@chromium.org> Fri Apr 26 14:14:08 2019 +0000
committer Commit Bot service account <commit-bot@chromium.org> Fri Apr 26 14:14:08 2019 +0000
tree 5ef25a1cfd483c5ba95bf2951fdcfea8205f3c4e
parent 15d4c2e63b7281d622eba063c615be73358af260

Metal: Invert CW and CCW because we compile with flip Y

BUG=dawn:43

Change-Id: I6d09118d141046cd307604b1b4d53a82d8d15bf6
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/6761
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>

src/dawn_native/metal/RenderPipelineMTL.mm

diff --git a/src/dawn_native/metal/RenderPipelineMTL.mm b/src/dawn_native/metal/RenderPipelineMTL.mm
index 0397ba6..b96416c 100644
--- a/src/dawn_native/metal/RenderPipelineMTL.mm
+++ b/src/dawn_native/metal/RenderPipelineMTL.mm
@@ -283,10 +283,11 @@
         }
 
         MTLWinding MTLFrontFace(dawn::FrontFace face) {
+            // Note that these are inverted because we flip the Y coordinate in the vertex shader
             switch (face) {
-                case dawn::FrontFace::CCW:
-                    return MTLWindingCounterClockwise;
                 case dawn::FrontFace::CW:
+                    return MTLWindingCounterClockwise;
+                case dawn::FrontFace::CCW:
                     return MTLWindingClockwise;
             }
         }