commit 560232a6ccf18d08aadb64cf2fd3349f789813ff
author Alexis Hetu <sugoi@google.com> Fri Sep 29 13:13:14 2017 -0400
committer Alexis Hétu <sugoi@google.com> Fri Sep 29 18:33:38 2017 +0000
tree 9b4d141a228baaa44aa4854cfaf632580c1e3aa6
parent 642fc923f8b7ef7bb0012583ca07d1e5971b7184

Handle WindowSurface::checkForResize() errors gracefully

WindowSurface::checkForResize() contained an assert which got
triggered in some tests. In order to avoid this issue, this cl
gracefully handles an invalid window handle.

Change-Id: I6bace88370980e94676d4af9de0d88e561175835
Reviewed-on: https://swiftshader-review.googlesource.com/12808
Tested-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>

src/OpenGL/libEGL/Surface.cpp

diff --git a/src/OpenGL/libEGL/Surface.cpp b/src/OpenGL/libEGL/Surface.cpp
index 8cb3b39..b3616b1 100644
--- a/src/OpenGL/libEGL/Surface.cpp
+++ b/src/OpenGL/libEGL/Surface.cpp
@@ -278,8 +278,7 @@
 		RECT client;
 		if(!GetClientRect(window, &client))
 		{
-			ASSERT(false);
-			return false;
+			return error(EGL_BAD_NATIVE_WINDOW, false);
 		}
 
 		int windowWidth = client.right - client.left;