Commit f71283c9 by Jamie Madill Committed by Commit Bot

Fix EGLContextCompatibilityTest.

The check for "IsPlatformAvailable" was inverted. We were incorrectly trying only to create a context on invalid configs. Bug: angleproject:4449 Change-Id: I03fbdc7794a338a55b9dc64521915a8be247bf35 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2376045 Commit-Queue: Jamie Madill <jmadill@chromium.org> Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org>
parent 776538aa
...@@ -71,6 +71,10 @@ bool ShouldSkipConfig(EGLDisplay display, EGLConfig config, bool windowSurfaceTe ...@@ -71,6 +71,10 @@ bool ShouldSkipConfig(EGLDisplay display, EGLConfig config, bool windowSurfaceTe
return windowSurfaceTest; return windowSurfaceTest;
} }
// Linux failures: http://anglebug.com/4990
if (IsLinux())
return true;
return false; return false;
} }
...@@ -489,7 +493,7 @@ void RegisterContextCompatibilityTests() ...@@ -489,7 +493,7 @@ void RegisterContextCompatibilityTests()
for (EGLint renderer : renderers) for (EGLint renderer : renderers)
{ {
PlatformParameters params = FromRenderer(renderer); PlatformParameters params = FromRenderer(renderer);
if (IsPlatformAvailable(params)) if (!IsPlatformAvailable(params))
continue; continue;
EGLint dispattrs[] = {EGL_PLATFORM_ANGLE_TYPE_ANGLE, renderer, EGL_NONE}; EGLint dispattrs[] = {EGL_PLATFORM_ANGLE_TYPE_ANGLE, renderer, EGL_NONE};
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment