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

Unified Diff: content/renderer/media/webmediaplayer_ms.cc

Issue 480233007: Remove implicit conversions from scoped_refptr to T* in content/renderer/media/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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
« no previous file with comments | « content/renderer/media/webmediaplayer_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/webmediaplayer_ms.cc
diff --git a/content/renderer/media/webmediaplayer_ms.cc b/content/renderer/media/webmediaplayer_ms.cc
index efd4f319a34919a8d65055e4849f359c6473f534..b8ecc146c2f99bb5452c754026125983d1fb4313 100644
--- a/content/renderer/media/webmediaplayer_ms.cc
+++ b/content/renderer/media/webmediaplayer_ms.cc
@@ -210,7 +210,7 @@ void WebMediaPlayerMS::pause() {
media_log_->AddEvent(media_log_->CreateEvent(media::MediaLogEvent::PAUSE));
- if (!current_frame_)
+ if (!current_frame_.get())
return;
// Copy the frame so that rendering can show the last received frame.
@@ -428,7 +428,7 @@ void WebMediaPlayerMS::OnFrameAvailable(
SetReadyState(WebMediaPlayer::ReadyStateHaveEnoughData);
GetClient()->sizeChanged();
- if (video_frame_provider_) {
+ if (video_frame_provider_.get()) {
video_weblayer_.reset(new cc_blink::WebLayerImpl(
cc::VideoLayer::Create(this, media::VIDEO_ROTATION_0)));
video_weblayer_->setOpaque(true);
« no previous file with comments | « content/renderer/media/webmediaplayer_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698