Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1111)

Unified Diff: content/renderer/gpu/gpu_video_service_host.cc

Issue 7361010: Enable fire-and-forget Destroy of HW video decoder, and misc other improvements. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: vrk CR responses. Created 9 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/gpu/gpu_video_service_host.cc
diff --git a/content/renderer/gpu/gpu_video_service_host.cc b/content/renderer/gpu/gpu_video_service_host.cc
deleted file mode 100644
index cb4333211cb844c1f1acd5adeff4eaa69e5cb910..0000000000000000000000000000000000000000
--- a/content/renderer/gpu/gpu_video_service_host.cc
+++ /dev/null
@@ -1,76 +0,0 @@
-// 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_service_host.h"
-
-#include "content/common/gpu/gpu_messages.h"
-#include "content/renderer/gpu/gpu_video_decode_accelerator_host.h"
-#include "content/renderer/render_thread.h"
-#include "media/video/video_decode_accelerator.h"
-
-GpuVideoServiceHost::GpuVideoServiceHost()
- : channel_(NULL),
- next_decoder_host_id_(0) {
- DCHECK(RenderThread::current());
- DCHECK_EQ(RenderThread::current()->message_loop(), MessageLoop::current());
-}
-
-GpuVideoServiceHost::~GpuVideoServiceHost() {
-}
-
-void GpuVideoServiceHost::set_channel(IPC::SyncChannel* channel) {
- DCHECK(CalledOnValidThread());
- DCHECK(!channel_);
- channel_ = channel;
- if (!on_initialized_.is_null())
- on_initialized_.Run();
-}
-
-bool GpuVideoServiceHost::OnMessageReceived(const IPC::Message& msg) {
- DCHECK(CalledOnValidThread());
- if (!channel_)
- return false;
- switch (msg.type()) {
- case AcceleratedVideoDecoderHostMsg_BitstreamBufferProcessed::ID:
- case AcceleratedVideoDecoderHostMsg_ProvidePictureBuffers::ID:
- case AcceleratedVideoDecoderHostMsg_InitializeDone::ID:
- case AcceleratedVideoDecoderHostMsg_DismissPictureBuffer::ID:
- case AcceleratedVideoDecoderHostMsg_PictureReady::ID:
- case AcceleratedVideoDecoderHostMsg_FlushDone::ID:
- case AcceleratedVideoDecoderHostMsg_ResetDone::ID:
- case AcceleratedVideoDecoderHostMsg_DestroyDone::ID:
- case AcceleratedVideoDecoderHostMsg_EndOfStream::ID:
- case AcceleratedVideoDecoderHostMsg_ErrorNotification::ID:
- if (router_.RouteMessage(msg))
- return true;
- LOG(ERROR) << "AcceleratedVideoDecoderHostMsg cannot be dispatched.";
- default:
- return false;
- }
-}
-
-void GpuVideoServiceHost::OnChannelError() {
- DCHECK(CalledOnValidThread());
- channel_ = NULL;
-}
-
-void GpuVideoServiceHost::SetOnInitialized(
- const base::Closure& on_initialized) {
- DCHECK(CalledOnValidThread());
- DCHECK(on_initialized_.is_null());
- on_initialized_ = on_initialized;
- if (channel_)
- on_initialized.Run();
-}
-
-GpuVideoDecodeAcceleratorHost* GpuVideoServiceHost::CreateVideoAccelerator(
- media::VideoDecodeAccelerator::Client* client,
- int32 command_buffer_route_id,
- gpu::CommandBufferHelper* cmd_buffer_helper) {
- DCHECK(CalledOnValidThread());
- DCHECK(channel_);
- return new GpuVideoDecodeAcceleratorHost(
- &router_, channel_, next_decoder_host_id_++,
- command_buffer_route_id, cmd_buffer_helper, client);
-}

Powered by Google App Engine
This is Rietveld 408576698