Enable fire-and-forget Destroy of HW video decoder, and misc other improvements.

content/renderer/gpu/command_buffer_proxy.cc

 // Copyright (c) 2011 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 "content/renderer/gpu/command_buffer_proxy.h"
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/process_util.h"
 #include "base/shared_memory.h"
@@ skipping 32 matching lines @@
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(CommandBufferProxy, message)
     IPC_MESSAGE_HANDLER(GpuCommandBufferMsg_UpdateState, OnUpdateState);
     IPC_MESSAGE_HANDLER(GpuCommandBufferMsg_Destroyed, OnDestroyed);
     IPC_MESSAGE_HANDLER(GpuCommandBufferMsg_SwapBuffers, OnSwapBuffers);
     IPC_MESSAGE_HANDLER(GpuCommandBufferMsg_NotifyRepaint,
                         OnNotifyRepaint);
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
 
-  if (!handled && video_decoder_host_.get())
+  if (!handled && video_decoder_host_)
     handled = video_decoder_host_->OnMessageReceived(message);
 
   DCHECK(handled);
   return handled;
 }
 
 void CommandBufferProxy::OnChannelError() {
   video_decoder_host_->OnChannelError();
   OnDestroyed(gpu::error::kUnknown);
 }
@@ skipping 315 matching lines @@
   if (last_state_.error != gpu::error::kNoError)
     return;
 
   Send(new GpuCommandBufferMsg_ResizeOffscreenFrameBuffer(route_id_, size));
 }
 
 void CommandBufferProxy::SetNotifyRepaintTask(Task* task) {
   notify_repaint_task_.reset(task);
 }
 
-GpuVideoDecodeAcceleratorHost* CommandBufferProxy::CreateVideoDecoder(
+scoped_refptr<GpuVideoDecodeAcceleratorHost>
+CommandBufferProxy::CreateVideoDecoder(
     const std::vector<uint32>& configs,
     gpu::CommandBufferHelper* cmd_buffer_helper,
     media::VideoDecodeAccelerator::Client* client) {
-  video_decoder_host_.reset(new GpuVideoDecodeAcceleratorHost(
-      channel_, route_id_, cmd_buffer_helper, client));
+  video_decoder_host_ = new GpuVideoDecodeAcceleratorHost(
+      channel_, route_id_, cmd_buffer_helper, client);
 
   if (!Send(new GpuCommandBufferMsg_CreateVideoDecoder(route_id_, configs))) {
     LOG(ERROR) << "Send(GpuChannelMsg_CreateVideoDecoder) failed";
-    video_decoder_host_.reset();
+    video_decoder_host_ = NULL;
   }
 
-  return video_decoder_host_.get();
+  return video_decoder_host_;
 }
 
 #if defined(OS_MACOSX)
 void CommandBufferProxy::SetWindowSize(const gfx::Size& size) {
   if (last_state_.error != gpu::error::kNoError)
     return;
 
   Send(new GpuCommandBufferMsg_SetWindowSize(route_id_, size));
 }
 #endif
@@ skipping 19 matching lines @@
   delete msg;
   return false;
 }
 
 void CommandBufferProxy::OnUpdateState(const gpu::CommandBuffer::State& state) {
   // Handle wraparound. It works as long as we don't have more than 2B state
   // updates in flight across which reordering occurs.
   if (state.generation - last_state_.generation < 0x80000000U)
     last_state_ = state;
 }