From 6f71fe8fcc06145939bcdf31d7ac2e9140fffa3f Mon Sep 17 00:00:00 2001 From: Anton Vayvod Date: Tue, 24 Jan 2017 09:55:36 -0800 Subject: [PATCH] [Video] Remove wrong DCHECK from EnableVideoTrackIfNeeded The DCHECK doesn't make sense as we might reenable the track during suspend/resume while in the background. BUG=683152 TEST=none Review-Url: https://codereview.chromium.org/2641373002 Cr-Commit-Position: refs/heads/master@{#445128} (cherry picked from commit fd01a87860fe7f105d25135a9ca0f8c3ecd9bdda) Review-Url: https://codereview.chromium.org/2651533007 . Cr-Commit-Position: refs/branch-heads/2987@{#58} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} --- media/blink/webmediaplayer_impl.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc index 71cfc3580dc8..e94d5e06b05c 100644 --- a/media/blink/webmediaplayer_impl.cc +++ b/media/blink/webmediaplayer_impl.cc @@ -2181,8 +2181,6 @@ bool WebMediaPlayerImpl::IsBackgroundOptimizationCandidate() const { } void WebMediaPlayerImpl::EnableVideoTrackIfNeeded() { - DCHECK(!IsHidden()); - // Don't change video track while the pipeline is resuming or seeking. if (is_pipeline_resuming_ || seeking_) return;