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

ppapi/shared_impl/ppb_video_decoder_shared.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ppapi/shared_impl/ppb_video_decoder_shared.h"
 
 #include "base/logging.h"
 #include "gpu/command_buffer/client/gles2_implementation.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/shared_impl/ppb_graphics_3d_shared.h"
@@ skipping 26 matching lines @@
     PP_Resource graphics_context,
     gpu::gles2::GLES2Implementation* gles2_impl) {
   DCHECK(graphics_context);
   DCHECK(!gles2_impl_ && !graphics_context_);
   gles2_impl_ = gles2_impl;
   PpapiGlobals::Get()->GetResourceTracker()->AddRefResource(graphics_context);
   graphics_context_ = graphics_context;
 }
 
 void PPB_VideoDecoder_Shared::Destroy() {
+  if (graphics_context_) {
+    PpapiGlobals::Get()->GetResourceTracker()->ReleaseResource(
+        graphics_context_);
+  }
   graphics_context_ = 0;

[piman, 2013/03/27 22:48:31]

nit: you can move this inside the if.

[danakj, 2013/03/27 22:54:52]

Done.

   gles2_impl_ = NULL;
-  PpapiGlobals::Get()->GetResourceTracker()->ReleaseResource(graphics_context_);
 }
 
 bool PPB_VideoDecoder_Shared::SetFlushCallback(
     scoped_refptr<TrackedCallback> callback) {
   if (TrackedCallback::IsPending(flush_callback_))
     return false;
   flush_callback_ = callback;
   return true;
 }
 
@@ skipping 37 matching lines @@
         graphics_context_, false);
     DCHECK(enter_g3d.succeeded());
     PPB_Graphics3D_Shared* graphics3d =
         static_cast<PPB_Graphics3D_Shared*>(enter_g3d.object());
     PPB_Graphics3D_Shared::ScopedNoLocking dont_lock(graphics3d);
     gles2_impl_->Flush();
   }
 }
 
 }  // namespace ppapi