commit 51706eaeeb4e1d90b92b6671398896de50b3157b
author Corentin Wallez <cwallez@chromium.org> Fri Aug 07 14:39:22 2015 -0400
committer Corentin Wallez <cwallez@chromium.org> Mon Aug 17 19:50:32 2015 +0000
tree 1a8b5fe5840b6611e8bc111b72166e0e7f314304
parent ada9ecc3ea8c2b8f169c61e1ec289b70b3282ea1

Make FramebufferAttachmentObject not refcountable

Re-land with a fix for an unitialized variable

Instead the refcount is done via callbacks. This allows Surface to
ignore this refcounting which will be useful in a follow-up CL.

BUG=angleproject:891

Change-Id: I1925ccaa4ce7b502b33088660d31c404b8313cb5
Reviewed-on: https://chromium-review.googlesource.com/293712
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Corentin Wallez <cwallez@chromium.org>

src/libANGLE/renderer/DisplayImpl.cpp

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