Pass Client pointer in Initialize() for VDA/VEA

media/filters/gpu_video_decoder.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 "media/filters/gpu_video_decoder.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
@@ skipping 172 matching lines @@
   needs_bitstream_conversion_ = (config.codec() == kCodecH264);
 
   if (previously_initialized) {
     // Reinitialization with a different config (but same codec and profile).
     // VDA should handle it by detecting this in-stream by itself,
     // no need to notify it.
     status_cb.Run(PIPELINE_OK);
     return;
   }
 
-  vda_ =
-      factories_->CreateVideoDecodeAccelerator(config.profile(), this).Pass();
+  vda_ = factories_->CreateVideoDecodeAccelerator(config.profile()).Pass();
   if (!vda_) {
     status_cb.Run(DECODER_ERROR_NOT_SUPPORTED);
     return;
   }
+  vda_->Initialize(this, config.profile());

[Ami GONE FROM CHROMIUM, 2014/02/24 23:12:40]

this can return false according to vda.h, but not according to gvdah.cc.  CHECK
it.

[sheu, 2014/02/24 23:48:20]

I'm going to fold it into the previous check.  I'd like to avoid the implicit
assumption that this is a GVDAH at this layer.

 
   DVLOG(3) << "GpuVideoDecoder::Initialize() succeeded.";
   media_log_->SetStringProperty("video_decoder", "gpu");
   status_cb.Run(PIPELINE_OK);
 }
 
 void GpuVideoDecoder::DestroyPictureBuffers(PictureBufferMap* buffers) {
   DCheckGpuVideoAcceleratorFactoriesTaskRunnerIsCurrent();
   for (PictureBufferMap::iterator it = buffers->begin(); it != buffers->end();
        ++it) {
@@ skipping 437 matching lines @@
     return;
   }
 }
 
 void GpuVideoDecoder::DCheckGpuVideoAcceleratorFactoriesTaskRunnerIsCurrent()
     const {
   DCHECK(factories_->GetTaskRunner()->BelongsToCurrentThread());
 }
 
 }  // namespace media