[Webkit-unassigned] [Bug 186971] Paused video improperly inhibits sleep

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=186971

--- Comment #5 from Eric Carlson <eric.carlson at apple.com> ---
(In reply to Philippe Normand from comment #4)
> (In reply to Philippe Normand from comment #2)
> > The problem is two-fold:
> > 
> > - the visibility of the media element changed but
> > HTMLMediaElement::shouldDisableSleep() returns too early the None result,
> > before checking the m_elementIsHidden variable.
> > 
> 
> Eric, would it be acceptable to move the the m_elementIsHidden test up in
> HTMLMediaElement::shouldDisableSleep() ?

We don't want to enable sleep for a "playing" that doesn't have audio and video, so where would we move the m_elementIsHidden test?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180625/c09f48d5/attachment.html>