Flash: Don't lose a reference to the Graphics3D from the VideoDecoder.

ppapi/shared_impl/ppb_video_decoder_shared.cc

Index: ppapi/shared_impl/ppb_video_decoder_shared.cc
diff --git a/ppapi/shared_impl/ppb_video_decoder_shared.cc b/ppapi/shared_impl/ppb_video_decoder_shared.cc
index ffd422d09979758f42fc6db044c9b1c5540d69d3..3a2bb42fdd034cfaf40d18966d2152cfd1aacd97 100644
--- a/ppapi/shared_impl/ppb_video_decoder_shared.cc
+++ b/ppapi/shared_impl/ppb_video_decoder_shared.cc
@@ -44,9 +44,12 @@ void PPB_VideoDecoder_Shared::InitCommon(
 }
 
 void PPB_VideoDecoder_Shared::Destroy() {
-  graphics_context_ = 0;
+  if (graphics_context_) {
+    PpapiGlobals::Get()->GetResourceTracker()->ReleaseResource(
+        graphics_context_);

[dmichael (off chromium), 2013/03/27 22:59:04]

What about either using ScopedPPResource, or actually getting the real
PPB_Graphics3D_Shared and holding a scoped_refptr to it?

[danakj, 2013/03/27 23:02:33]

Will ScopedPPResource work with this model where Destroy() is being called
manually? That would move the release to the object's destructor. But the proxy
seems to need to call this function directly.

[dmichael (off chromium), 2013/03/28 02:40:49]

Didn't see that ScopedPPResource has no public way to release a ref.

optional: having graphics_context_ be a scoped_refptr<PPB_Graphics3D_Shared>
instead might actually be pretty clean, and protects you from the plugin doing
stupid stuff with the refcount. But with this being a private API, it's not
important. It's kind of irrelevant except it guarantees the Graphics3D gets
released even if Destroy is not called.

[danakj, 2013/03/28 15:58:50]

I feel hesitant to make that change as it would be pretty invasive and this
needs to be backported to the branch. The graphics context resource id is
plumbed through at least 3 or 4 files at the moment..

I do like your point though, I will add a DCHECK to the destructor to help us
verify Destroy() continues to be called.

+    graphics_context_ = 0;
+  }
   gles2_impl_ = NULL;
-  PpapiGlobals::Get()->GetResourceTracker()->ReleaseResource(graphics_context_);
 }
 
 bool PPB_VideoDecoder_Shared::SetFlushCallback(