New issue
Advanced search Search tips
Starred by 1 user
Status: Fixed
Closed: Mar 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Sign in to add a comment
Replace MediaWebContentsObserver::HasActiveEffectivelyFullscreenVideo DCHECK with early return
Project Member Reported by, Mar 17 2017 Back to list
The only caller for this method has changed and no longer check for fullscreen, fully relying on this method to return false when not fullscreen. Updating the fullscreen check to follow that reality.
Project Member Comment 1 by, Mar 20 2017
The following revision refers to this bug:

commit fc7f323f548c926b99fe6ebd15f46a514f0df198
Author: mlamouri <>
Date: Mon Mar 20 20:25:24 2017

Media: replace HasActiveEffectivelyFullscreenVideo DCHECK with early return.

BUG= 702647

Cr-Commit-Position: refs/heads/master@{#458167}


Labels: -M-58 M-59
Status: Fixed
No need to merge this, it should have no impact on release builds.
Comment 3 by, Mar 29 2017
Blocking: -679364
Sign in to add a comment