commit 18fdcbcf1d02d3b3b4b5c712f05058f2e8d629c6
author Jamie Madill <jmadill@chromium.org> Wed Aug 19 18:12:44 2015 +0000
committer Jamie Madill <jmadill@chromium.org> Wed Aug 19 18:23:33 2015 +0000
tree 776bbe7ff56416fbe6e91870ac564d6b0d9569b3
parent 1661fde6c4e528679e85a04557fbbeb5d24e5a7c

Revert "Make the default framebuffer owned by Surface"

Seems to be causing failures on the Mac GPU FYI bots.
SurfaceTest.DestructionDeletesImpl seems to crash.

BUG=angleproject:891
BUG=522557

This reverts commit 264ab56f2e70431e8310f9353952acd3b22b466f.

Change-Id: I80aeecb8e191de011d9afe6534d0285dcffa82a3
Reviewed-on: https://chromium-review.googlesource.com/294540
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

src/libANGLE/renderer/DisplayImpl.cpp

diff --git a/src/libANGLE/renderer/DisplayImpl.cpp b/src/libANGLE/renderer/DisplayImpl.cpp
index 2eb44eb..16446f9 100644
--- a/src/libANGLE/renderer/DisplayImpl.cpp
+++ b/src/libANGLE/renderer/DisplayImpl.cpp
@@ -29,8 +29,8 @@
 
 void DisplayImpl::destroySurface(egl::Surface *surface)
 {
-    mSurfaceSet.erase(surface);
     surface->onDestroy();
+    mSurfaceSet.erase(surface);
 }
 
 const egl::DisplayExtensions &DisplayImpl::getExtensions() const