commit c6faa9745f889b838ec8a7c6ad255bbb1c51f48d
author Yuly Novikov <ynovikov@chromium.org> Wed Dec 20 19:50:10 2017 -0500
committer Commit Bot <commit-bot@chromium.org> Thu Dec 21 01:21:37 2017 +0000
tree 078e90a52c45e24d78b719cf658f94a1ed3b87c6
parent 0c37100dfc684824ee3c532d90c8f2e3f03e10e2

Skip UniformBufferTest.SimpleBindingChange on Mac NVIDIA GL

Unskip UniformBufferTest.SimpleBufferChange,
which was skipped by mistake instead of this test in
https://chromium-review.googlesource.com/833287

BUG=angleproject:2287

Change-Id: Id79821b26ad38ed07d1193b717ad222df3bb58fe
Reviewed-on: https://chromium-review.googlesource.com/838262
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Commit-Queue: Yuly Novikov <ynovikov@chromium.org>

src/tests/gl_tests/UniformBufferTest.cpp

diff --git a/src/tests/gl_tests/UniformBufferTest.cpp b/src/tests/gl_tests/UniformBufferTest.cpp
index 52f6fe9..dade904 100644
--- a/src/tests/gl_tests/UniformBufferTest.cpp
+++ b/src/tests/gl_tests/UniformBufferTest.cpp
@@ -1414,6 +1414,9 @@
 // Regression test for a dirty bit bug in ANGLE. See http://crbug.com/792966
 TEST_P(UniformBufferTest, SimpleBindingChange)
 {
+    // http://anglebug.com/2287
+    ANGLE_SKIP_TEST_IF(IsOSX() && IsNVIDIA() && IsDesktopOpenGL());
+
     ANGLE_GL_PROGRAM(program, kVertexShader, kFragmentShader);
 
     glBindAttribLocation(program, 0, "a_vertex");
@@ -1459,9 +1462,6 @@
 // Regression test for a dirty bit bug in ANGLE. Same as above but for the indexed bindings.
 TEST_P(UniformBufferTest, SimpleBufferChange)
 {
-    // anglebug.com/2287
-    ANGLE_SKIP_TEST_IF(IsOSX() && IsNVIDIA() && IsDesktopOpenGL());
-
     ANGLE_GL_PROGRAM(program, kVertexShader, kFragmentShader);
 
     glBindAttribLocation(program, 0, "a_vertex");