Skip to content

Commit

Permalink
[Video] Remove wrong DCHECK from EnableVideoTrackIfNeeded
Browse files Browse the repository at this point in the history
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 fd01a87)

Review-Url: https://codereview.chromium.org/2651533007 .
Cr-Commit-Position: refs/branch-heads/2987@{#58}
Cr-Branched-From: ad51088-refs/heads/master@{#444943}
  • Loading branch information
Anton Vayvod committed Jan 24, 2017
1 parent 47a2344 commit 6f71fe8
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions media/blink/webmediaplayer_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 6f71fe8

Please sign in to comment.