Fix a compile error in SwapChain11.cpp

In function resetOffscreenColorBuffer, the local variable 'result' is never used in its scope, so when attempting to build a release version of Chromium with latest ANGLE on Windows, ninja will report a compile warning and stop the compilation: 'result': local variable is initialized but not referenced. The build failure starts at: https://chromium-review.googlesource.com/#/c/316804/ Change-Id: I29c05653f0d2e4f34c2e0402848f403ec2af8e5a Reviewed-on: https://chromium-review.googlesource.com/404515Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Geoff Lang <geofflang@chromium.org>

Fix a compile error in SwapChain11.cpp
bc78fc61 · Jiawei-Shao · Commit Bot · 77c13aaf · bc78fc61
Commit bc78fc61 authored Oct 28, 2016 by Jiawei-Shao Committed by Commit Bot Oct 28, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 1 deletion

SwapChain11.cpp src/libANGLE/renderer/d3d/d3d11/SwapChain11.cpp +6 -1

No files found.
--- a/src/libANGLE/renderer/d3d/d3d11/SwapChain11.cpp
+++ b/src/libANGLE/renderer/d3d/d3d11/SwapChain11.cpp
@@ -204,7 +204,12 @@ EGLint SwapChain11::resetOffscreenColorBuffer(int backbufferWidth, int backbuffe
            ID3D11Resource *tempResource11;
            HRESULT result = device->OpenSharedResource(mShareHandle, __uuidof(ID3D11Resource),
                                                        (void **)&tempResource11);
-            ASSERT(SUCCEEDED(result));
+            if (FAILED(result))
+            {
+                ERR("Failed to open the swap chain pbuffer share handle: %08lX", result);
+                release();
+                return EGL_BAD_PARAMETER;
+            }

            mOffscreenTexture = d3d11::DynamicCastComObject<ID3D11Texture2D>(tempResource11);
            SafeRelease(tempResource11);