Pass Client pointer in Initialize() for VDA/VEA

content/renderer/media/rtc_video_decoder.cc

Index: content/renderer/media/rtc_video_decoder.cc
diff --git a/content/renderer/media/rtc_video_decoder.cc b/content/renderer/media/rtc_video_decoder.cc
index ce1f24c6b520755daf9a58e07f2a2a7fd7265be7..78be1a60b823c2281c83a2ca038970d7905f26c8 100644
--- a/content/renderer/media/rtc_video_decoder.cc
+++ b/content/renderer/media/rtc_video_decoder.cc
@@ -672,7 +672,9 @@ void RTCVideoDecoder::ReusePictureBuffer(
 void RTCVideoDecoder::CreateVDA(media::VideoCodecProfile profile,
                                 base::WaitableEvent* waiter) {
   DCHECK(vda_task_runner_->BelongsToCurrentThread());
-  vda_ = factories_->CreateVideoDecodeAccelerator(profile, this);
+  vda_ = factories_->CreateVideoDecodeAccelerator(profile);
+  if (vda_)
+    vda_->Initialize(this, profile);

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

CHECK this doesn't fail (b/c gvdah.cc can't)

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

The call point for CreateVDA() is in RTCVideoDecoder::Create().  Constructor
failure is checked for there; I guess we can fold this in to have a more useful
failure path than just a CHECK().

   waiter->Signal();
 }