RESOLVED DUPLICATE of bug 127701127758
Crash in MediaPlayerPrivateAVFoundationObjC::seekToTime dereferencing RefPtr from invalid thread. 
https://bugs.webkit.org/show_bug.cgi?id=127758
Summary Crash in MediaPlayerPrivateAVFoundationObjC::seekToTime dereferencing RefPtr ...
Jeremy Jones
Reported 2014-01-27 20:27:19 PST
Crash in MediaPlayerPrivateAVFoundationObjC::seekToTime dereferencing RefPtr from invalid thread.
Attachments
Patch (2.20 KB, patch)
2014-01-27 20:30 PST, Jeremy Jones 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jeremy Jones
Comment 1 2014-01-27 20:30:42 PST
Created attachment 222400 [details] Patch
Jer Noble
Comment 2 2014-01-27 20:34:34 PST
Looks like this was fixed by bug #127701. In the future, WebThreadRun() is IOS only, but callOnMainThread() will cause the passed function to be called on the web-thread (regardless of it's name).
Darin Adler
Comment 3 2014-01-28 10:05:45 PST
Comment on attachment 222400 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=222400&action=review Should upload a new patch. EWS was unable to apply this one. > Source/WebCore/ChangeLog:8 > + A block is dispatched to the main thread; dispatch it again to the webthread. Can’t we go straight to the web thread? Why do we need both?
Jer Noble
Comment 4 2014-01-28 10:21:30 PST
(In reply to comment #3) > (From update of attachment 222400 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=222400&action=review > > Should upload a new patch. EWS was unable to apply this one. > > > Source/WebCore/ChangeLog:8 > > + A block is dispatched to the main thread; dispatch it again to the webthread. > > Can’t we go straight to the web thread? Why do we need both? Indeed, and that's what we did in bug #127701. Marking this bug as a dup of that one. *** This bug has been marked as a duplicate of bug 127701 ***
Brent Fulgham
Comment 5 2014-02-05 16:38:33 PST
Comment on attachment 222400 [details] Patch Clearing patch flag since this was closed as a duplicate.
Note You need to log in before you can comment on or make changes to this bug.