Add play state callbacks to MediaControls, kill timeUpdate timer.
Created attachment 85944 [details] Patch
Comment on attachment 85944 [details] Patch Nice!
(In reply to comment #2) > (From update of attachment 85944 [details]) > Nice! Since this change does an extra update of the timeline/status as soon as we start playing, the status values are more accurate. However, this affects results in media/controls-without-preload.html, because canplaythrough event is fired asynchronously. By the time it's fired, we already started playing and the status is slightly different. Should I: a) change expectations for the test b) remove updating on start?
(In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 85944 [details] [details]) > > Nice! > > Since this change does an extra update of the timeline/status as soon as we start playing, the status values are more accurate. However, this affects results in media/controls-without-preload.html, because canplaythrough event is fired asynchronously. By the time it's fired, we already started playing and the status is slightly different. > > Should I: > a) change expectations for the test > b) remove updating on start? I ended up improving the test in http://trac.webkit.org/changeset/81280.
Committed r81283: <http://trac.webkit.org/changeset/81283>