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

Unified Diff: media/blink/webmediaplayer_impl.cc

Issue 2508053002: media: Do a TimedWait() for video surface teardown in AVDA (Closed)
Patch Set: cleanup/couple more tests Created 4 years, 1 month 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/blink/webmediaplayer_impl.cc
diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
index 32d5a896e44b37814eeb5ad2fc33cdfc523e9075..a63c245714dcedfeac5fc5654a630be510185b23 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -1503,6 +1503,7 @@ void WebMediaPlayerImpl::NotifyDownloading(bool is_downloading) {
}
void WebMediaPlayerImpl::OnSurfaceCreated(int surface_id) {
+ DVLOG(1) << __func__;
overlay_surface_id_ = surface_id;
if (!set_surface_cb_.is_null()) {
// If restart is required, the callback is one-shot only.
@@ -1519,6 +1520,7 @@ void WebMediaPlayerImpl::OnSurfaceRequested(
DCHECK(main_task_runner_->BelongsToCurrentThread());
DCHECK(surface_manager_);
DCHECK(!use_fallback_path_);
+ DVLOG(1) << __func__;
// A null callback indicates that the decoder is going away.
if (set_surface_cb.is_null()) {

Powered by Google App Engine
This is Rietveld 408576698