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

content/renderer/gpu/gpu_video_decode_accelerator_host.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/gpu_video_decode_accelerator_host.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/shared_memory.h"
@@ skipping 41 matching lines @@
     IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_ProvidePictureBuffers,
                         OnProvidePictureBuffer)
     IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_InitializeDone,
                         OnInitializeDone)
     IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_PictureReady,
                         OnPictureReady)
     IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_FlushDone,
                         OnFlushDone)
     IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_ResetDone,
                         OnResetDone)
-    IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_DestroyDone,
-                        OnDestroyDone)
     IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_EndOfStream,
                         OnEndOfStream)
     IPC_MESSAGE_HANDLER(AcceleratedVideoDecoderHostMsg_ErrorNotification,
                         OnErrorNotification)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   DCHECK(handled);
   return handled;
 }
 
@@ skipping 56 matching lines @@
 }
 
 void GpuVideoDecodeAcceleratorHost::Flush() {
   DCHECK(CalledOnValidThread());
   Send(new AcceleratedVideoDecoderMsg_Flush(
       command_buffer_route_id_, SyncTokens()));
 }
 
 void GpuVideoDecodeAcceleratorHost::Reset() {
   DCHECK(CalledOnValidThread());
-  if (!ipc_sender_->Send(new AcceleratedVideoDecoderMsg_Reset(
-          command_buffer_route_id_, SyncTokens()))) {
-    LOG(ERROR) << "Send(AcceleratedVideoDecoderMsg_Reset) failed";
-    // TODO(fischman/vrk): signal error to client.
-    return;
-  }
+  Send(new AcceleratedVideoDecoderMsg_Reset(
+      command_buffer_route_id_, SyncTokens()));
 }
 
 void GpuVideoDecodeAcceleratorHost::Destroy() {
   DCHECK(CalledOnValidThread());
   Send(new AcceleratedVideoDecoderMsg_Destroy(
       command_buffer_route_id_, SyncTokens()));
 }
 
 void GpuVideoDecodeAcceleratorHost::Send(IPC::Message* message) {
   // After OnChannelError is called, the client should no longer send
@@ skipping 46 matching lines @@
 void GpuVideoDecodeAcceleratorHost::OnFlushDone() {
   DCHECK(CalledOnValidThread());
   client_->NotifyFlushDone();
 }
 
 void GpuVideoDecodeAcceleratorHost::OnResetDone() {
   DCHECK(CalledOnValidThread());
   client_->NotifyResetDone();
 }
 
-void GpuVideoDecodeAcceleratorHost::OnDestroyDone() {
-  DCHECK(CalledOnValidThread());
-  client_->NotifyDestroyDone();
-}
-
 void GpuVideoDecodeAcceleratorHost::OnEndOfStream() {
   DCHECK(CalledOnValidThread());
   client_->NotifyEndOfStream();
 }
 
 void GpuVideoDecodeAcceleratorHost::OnErrorNotification(uint32 error) {
   DCHECK(CalledOnValidThread());
   client_->NotifyError(
       static_cast<media::VideoDecodeAccelerator::Error>(error));
 }