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

Unified Diff: media/filters/gpu_video_decoder.cc

Issue 16136005: Update remaining files to use WeakPtr<T>::get() instead of "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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 8cfcb67705a7cfb24e9dd299883655da6b7ac24f..956b4d5e9373138df918b1b528da38caac3117a4 100644
--- a/media/filters/gpu_video_decoder.cc
+++ b/media/filters/gpu_video_decoder.cc
@@ -69,14 +69,14 @@ VDAClientProxy::VDAClientProxy(VideoDecodeAccelerator::Client* client)
: client_loop_(base::MessageLoopProxy::current()),
weak_client_factory_(client),
weak_client_(weak_client_factory_.GetWeakPtr()) {
- DCHECK(weak_client_);
+ DCHECK(weak_client_.get());
}
VDAClientProxy::~VDAClientProxy() {}
void VDAClientProxy::Detach() {
DCHECK(client_loop_->BelongsToCurrentThread());
- DCHECK(weak_client_) << "Detach() already called";
+ DCHECK(weak_client_.get()) << "Detach() already called";
weak_client_factory_.InvalidateWeakPtrs();
}
@@ -306,9 +306,9 @@ void GpuVideoDecoder::DestroyTextures() {
static void DestroyVDAWithClientProxy(
const scoped_refptr<VDAClientProxy>& client_proxy,
base::WeakPtr<VideoDecodeAccelerator> weak_vda) {
- if (weak_vda) {
+ if (weak_vda.get()) {
weak_vda->Destroy();
- DCHECK(!weak_vda); // Check VDA::Destroy() contract.
+ DCHECK(!weak_vda.get()); // Check VDA::Destroy() contract.
}
}

Powered by Google App Engine
This is Rietveld 408576698