Bug 140468 - [Mac][MSE] Stalled YouTube playback does not allow display to sleep.
Summary: [Mac][MSE] Stalled YouTube playback does not allow display to sleep.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jer Noble
URL:
Keywords:
Depends on: 140611
Blocks:
  Show dependency tree / graph
 
Reported: 2015-01-14 14:50 PST by Jer Noble
Modified: 2016-04-30 11:00 PDT (History)
3 users (show)

See Also:

Attachments
Patch (890.86 KB, patch)
2015-01-14 14:55 PST, Jer Noble 		no flags Details | Formatted Diff | Diff
Patch for landing (892.12 KB, patch)
2015-01-14 15:22 PST, Jer Noble 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jer Noble 2015-01-14 14:50:38 PST 
[Mac][MSE] Stalled YouTube playback does not allow display to sleep.
Comment 1 Jer Noble 2015-01-14 14:55:27 PST 
Created attachment 244643 [details]
Patch
Comment 2 Eric Carlson 2015-01-14 15:03:10 PST 
Comment on attachment 244643 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=244643&action=review

> LayoutTests/media/media-source/media-source-stalled-holds-sleep-assertion.html:12
> +    function runTest() {

You know what I think of having the function's opening brace here...
Comment 3 Jer Noble 2015-01-14 15:22:20 PST 
Created attachment 244650 [details]
Patch for landing
Comment 4 WebKit Commit Bot 2015-01-14 22:00:48 PST 
Comment on attachment 244650 [details]
Patch for landing

Clearing flags on attachment: 244650

Committed r178486: <http://trac.webkit.org/changeset/178486>
Comment 5 Csaba Osztrogonác 2015-01-19 04:55:27 PST 
(In reply to comment #4)
> Comment on attachment 244650 [details]
> Patch for landing
> 
> Clearing flags on attachment: 244650
> 
> Committed r178486: <http://trac.webkit.org/changeset/178486>

It broke the !ENABLE(VIDEO) build, new bug report with the fix: bug140611 .
Comment 6 Simon Fraser (smfr) 2016-04-30 11:00:08 PDT 
This test is timing out everywhere:
https://build.webkit.org/results/Apple%20Yosemite%20Debug%20WK2%20(Tests)/r200289%20(11788)/results.html