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

Unified Diff: media/filters/gpu_video_decoder.cc

Issue 170843004: Pass Client pointer in Initialize() for VDA/VEA (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: b0ec4672 Build fixes. Created 6 years, 10 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: media/filters/gpu_video_decoder.cc
diff --git a/media/filters/gpu_video_decoder.cc b/media/filters/gpu_video_decoder.cc
index b87041a98b5db6126cdeca5859b64a303e8a0382..e2e45764e7067fda3da68d7bb5a5ac97713d957f 100644
--- a/media/filters/gpu_video_decoder.cc
+++ b/media/filters/gpu_video_decoder.cc
@@ -190,12 +190,12 @@ void GpuVideoDecoder::Initialize(const VideoDecoderConfig& config,
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
sheu 2014/02/24 23:48:20 I'm going to fold it into the previous check. I'd
DVLOG(3) << "GpuVideoDecoder::Initialize() succeeded.";
media_log_->SetStringProperty("video_decoder", "gpu");

Powered by Google App Engine
This is Rietveld 408576698