[Mac] Toggling in and out of Youtube videos w/ HTML5 player in Safari causes the video to start over
<rdar://problem/16382745>
Created attachment 228474 [details] Patch
Comment on attachment 228474 [details] Patch Attachment 228474 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/5277128169881600 New failing tests: media/video-load-preload-metadata.html media/media-continues-playing-after-replace-source.html media/video-restricted-no-preload-auto.html
Created attachment 228482 [details] Archive of layout-test-results from webkit-ews-08 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-08 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Comment on attachment 228474 [details] Patch Attachment 228474 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/4977464811978752 New failing tests: media/video-load-preload-metadata.html platform/mac/fast/scrolling/scroll-iframe-latched-mainframe.html platform/mac/fast/scrolling/scroll-select-latched-mainframe.html media/media-continues-playing-after-replace-source.html media/video-restricted-no-preload-auto.html platform/mac/fast/scrolling/scroll-div-latched-mainframe.html
Created attachment 228484 [details] Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-15 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Created attachment 228486 [details] Patch
Comment on attachment 228486 [details] Patch Attachment 228486 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/5234183597195264 New failing tests: media/media-fragments/TC0011.html media/media-fragments/TC0014.html media/video-canvas-drawing-output.html media/track/regions-webvtt/vtt-region-display.html media/track/track-css-all-cues.html media/track/track-css-cue-lifetime.html media/media-fragments/TC0038.html media/media-fragments/TC0006.html compositing/layers-inside-overflow-scroll.html media/track/track-css-matching-lang.html compositing/self-painting-layers.html compositing/overflow/scroll-ancestor-update.html fast/repaint/obscured-background-no-repaint.html compositing/geometry/clipped-video-controller.html platform/mac/fast/scrolling/scroll-iframe-latched-mainframe.html compositing/geometry/video-fixed-scrolling.html media/track/track-css-matching-default.html media/media-fragments/TC0039.html platform/mac/fast/scrolling/scroll-div-latched-mainframe.html media/media-fragments/TC0024.html compositing/video/video-poster.html platform/mac/fast/scrolling/scroll-select-latched-mainframe.html media/controls-drag-timebar.html media/media-fragments/TC0015.html media/media-fragments/TC0036.html media/video-canvas-drawing.html media/media-fragments/TC0005.html media/media-fragments/TC0009.html media/media-fragments/TC0037.html media/media-fragments/TC0035.html
Created attachment 228488 [details] Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-14 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Comment on attachment 228486 [details] Patch Attachment 228486 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/5485925589057536 New failing tests: media/media-fragments/TC0011.html media/media-fragments/TC0014.html media/video-canvas-drawing-output.html media/track/regions-webvtt/vtt-region-display.html media/track/track-css-all-cues.html media/track/track-css-matching-timestamps.html media/track/track-css-cue-lifetime.html media/media-fragments/TC0038.html media/media-fragments/TC0006.html compositing/layers-inside-overflow-scroll.html media/track/track-css-matching-lang.html compositing/self-painting-layers.html compositing/overflow/scroll-ancestor-update.html media/event-attributes.html compositing/geometry/video-fixed-scrolling.html media/media-fragments/TC0053.html media/track/track-css-matching-default.html media/media-fragments/TC0039.html media/media-fragments/TC0024.html compositing/video/video-poster.html media/controls-drag-timebar.html media/media-fragments/TC0015.html media/media-fragments/TC0036.html media/video-canvas-drawing.html media/media-fragments/TC0051.html media/media-fragments/TC0005.html media/media-fragments/TC0009.html media/media-fragments/TC0037.html media/media-fragments/TC0035.html media/media-fragments/TC0052.html
Created attachment 228495 [details] Archive of layout-test-results from webkit-ews-04 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-04 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Comment on attachment 228486 [details] Patch Attachment 228486 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/5860175080587264 New failing tests: media/media-fragments/TC0011.html media/media-fragments/TC0014.html media/video-canvas-drawing-output.html media/track/regions-webvtt/vtt-region-display.html media/track/track-css-all-cues.html media/track/track-css-matching-timestamps.html media/track/track-css-cue-lifetime.html media/media-fragments/TC0038.html media/media-fragments/TC0006.html compositing/layers-inside-overflow-scroll.html media/track/track-css-matching-lang.html compositing/self-painting-layers.html compositing/overflow/scroll-ancestor-update.html media/event-attributes.html compositing/geometry/video-fixed-scrolling.html media/media-fragments/TC0053.html media/track/track-css-matching-default.html media/media-fragments/TC0039.html media/media-fragments/TC0024.html compositing/video/video-poster.html media/controls-drag-timebar.html media/media-fragments/TC0015.html media/media-fragments/TC0036.html media/video-canvas-drawing.html media/media-fragments/TC0051.html media/media-fragments/TC0005.html media/media-fragments/TC0009.html media/media-fragments/TC0037.html media/media-fragments/TC0035.html media/media-fragments/TC0052.html
Created attachment 228504 [details] Archive of layout-test-results from webkit-ews-01 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-01 Port: mac-mountainlion Platform: Mac OS X 10.8.5
So far, the fixes for YouTube's behavior have broken more than they fix :-(
The last thing I want to try is to fetch the type of load from the AVAsset accessLog, and if it's a non-stream, always make the maximum seekable time == duration
Nope, the access log is a AVPlayerItem construct, so it won't exist at the point where YouTube is making its seek request.
YouTube needs to check the contents of video.seekable, and if that TimeRanges is empty, or has empty contents, it needs to wait till the 'loadeddata' event before checking again.
Comment on attachment 228486 [details] Patch Cleared review? from attachment 228486 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).