gpu: GpuVideo*Host stop hold ref pointer to GpuChannelHost.

Singleton is destructed at exit(), we are kind of late to release the last ref count for GpuChannelHost.
use raw pointer for now.
long term plan is to either forward close() to GpuVideo* or We move our singleton to destruct at ~RenderThread() where message loop is still alive?

BUG=53896
TEST=None.

Review URL: http://codereview.chromium.org/3232008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58114 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 4 deletions

diff --git a/chrome/renderer/gpu_video_service_host.cc b/chrome/renderer/gpu_video_service_host.cc
index 61a2dcd..c80e35d 100644
--- a/chrome/renderer/gpu_video_service_host.cc
+++ b/chrome/renderer/gpu_video_service_host.cc
@@ -9,14 +9,15 @@
 #include "chrome/renderer/render_thread.h"
 
 GpuVideoServiceHost::GpuVideoServiceHost()
-    : router_(NULL),
+    : channel_host_(NULL),
+      router_(NULL),
       message_loop_(NULL) {
   memset(&service_info_, 0, sizeof(service_info_));
 }
 
 void GpuVideoServiceHost::OnChannelError() {
   LOG(ERROR) << "GpuVideoServiceHost::OnChannelError";
-  channel_host_.release();
+  channel_host_ = NULL;
   router_ = NULL;
 }
 
@@ -32,7 +33,7 @@ scoped_refptr<GpuVideoDecoderHost> GpuVideoServiceHost::CreateVideoDecoder(
     GpuVideoDecoderHost::EventHandler* event_handler) {
   DCHECK(RenderThread::current());
 
-  if (!channel_host_.get() || !service_info_.service_available_)
+  if (!channel_host_ || !service_info_.service_available_)
     return NULL;
 
   GpuVideoDecoderInfoParam param;
@@ -60,7 +61,7 @@ void GpuVideoServiceHost::DestroyVideoDecoder(
     scoped_refptr<GpuVideoDecoderHost> gpu_video_decoder_host) {
   DCHECK(RenderThread::current());
 
-  if (!channel_host_.get() || !service_info_.service_available_)
+  if (!channel_host_ || !service_info_.service_available_)
     return;
 
   DCHECK(gpu_video_decoder_host.get());