RESOLVED FIXED 68056
MediaPlayerPrivateAVFoundationObjC's "periodic time observer" is unnecessary 
https://bugs.webkit.org/show_bug.cgi?id=68056
Summary MediaPlayerPrivateAVFoundationObjC's "periodic time observer" is unnecessary
Eric Carlson
Reported 2011-09-13 21:42:45 PDT
MediaPlayerPrivateAVFoundationObjC::createAVPlayer creates a periodic time observer that fires once in a very long while just so it can also be called whenever the time is changed. This was originally necessary early on, but AVPlayerItem's -seekToTime:toleranceBefore:toleranceAfter:completionHandler makes it completely unnecessary.
Attachments
Proposed patch. (2.82 KB, patch)
2011-09-13 22:07 PDT, Eric Carlson 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2011-09-13 21:43:37 PDT
<rdar://problem/10121225>
Eric Carlson
Comment 2 2011-09-13 22:07:12 PDT
Created attachment 107286 [details] Proposed patch.
Darin Adler
Comment 3 2011-09-13 22:14:46 PDT
Comment on attachment 107286 [details] Proposed patch. Patch would be better if it explained why we thought it was needed, what changed, and so why it’s not needed now.
Eric Carlson
Comment 4 2011-09-14 11:59:36 PDT
http://trac.webkit.org/changeset/95104
Note You need to log in before you can comment on or make changes to this bug.