Fix a bug in EGLPresentPathD3D11Test

This test was releasing a resource created with a queried device after it released the D3D11 device. It is problematic because the error was not caught by the swarming tests on the trybots, but did show up on the Intel bot because it was not a swarming test. BUG=589851 BUG=angleproject:1219 Change-Id: I7383640df9794aaf0e32e0a7c276ae3b22232050 Reviewed-on: https://chromium-review.googlesource.com/330001Reviewed-by: Jamie Madill <jmadill@chromium.org>

Fix a bug in EGLPresentPathD3D11Test
e6bcdd24 · Jamie Madill · 518b9fab · e6bcdd24
Commit e6bcdd24 authored Mar 02, 2016 by Jamie Madill
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

EGLPresentPathD3D11Test.cpp src/tests/egl_tests/EGLPresentPathD3D11Test.cpp +2 -2

No files found.
--- a/src/tests/egl_tests/EGLPresentPathD3D11Test.cpp
+++ b/src/tests/egl_tests/EGLPresentPathD3D11Test.cpp
@@ -151,6 +151,8 @@ class EGLPresentPathD3D11 : public testing::TestWithParam<PlatformParameters>

    void TearDown() override
    {
+        SafeRelease(mOffscreenSurfaceD3D11Texture);
+
        if (mDisplay != EGL_NO_DISPLAY)
        {
            eglMakeCurrent(mDisplay, EGL_NO_SURFACE, EGL_NO_SURFACE, EGL_NO_CONTEXT);
@@ -173,8 +175,6 @@ class EGLPresentPathD3D11 : public testing::TestWithParam<PlatformParameters>

        mOSWindow->destroy();
        SafeDelete(mOSWindow);
-
-        SafeRelease(mOffscreenSurfaceD3D11Texture);
    }

    void drawQuadUsingGL()