Bug 123545 - [Win] Legible Output callbacks should be on notification queue
Summary: [Win] Legible Output callbacks should be on notification queue
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2013-10-30 17:33 PDT by Brent Fulgham
Modified: 2013-10-30 17:48 PDT (History)
6 users (show)

See Also:

Attachments
Patch (5.45 KB, patch)
2013-10-30 17:41 PDT, Brent Fulgham 		eric.carlson: review+
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 Brent Fulgham 2013-10-30 17:33:53 PDT 
We should register our Legible Output notifications to run on the notification queue, rather than the main queue. If we don't we can sometimes run into shutdown problems where video playback does not end when we release the media resource because the main dispatch queue continues to run as long as the application those. This causes these resources to be held onto, resulting in media not stopping and preventing WebKit from exiting when the program is shut down.
Comment 1 Radar WebKit Bug Importer 2013-10-30 17:34:21 PDT 
<rdar://problem/15358129>
Comment 2 Brent Fulgham 2013-10-30 17:41:21 PDT 
Created attachment 215585 [details]
Patch
Comment 3 Brent Fulgham 2013-10-30 17:48:35 PDT 
Committed r158334: <http://trac.webkit.org/changeset/158334>