Bug 131154

Summary: [Mac] Toggling in and out of Youtube videos w/ HTML5 player in Safari causes the video to start over
Product: WebKit Reporter: Jer Noble <jer.noble>
Component: New BugsAssignee: Jer Noble <jer.noble>
Status: RESOLVED WONTFIX    
Severity: Normal CC: buildbot, commit-queue, eric.carlson, glenn, philipj, rniwa, sergio
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion
none
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2
none
Patch
none
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2
none
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion
none
Archive of layout-test-results from webkit-ews-01 for mac-mountainlion none

Jer Noble
Reported 2014-04-02 23:46:41 PDT
[Mac] Toggling in and out of Youtube videos w/ HTML5 player in Safari causes the video to start over
Attachments
Patch (2.62 KB, patch)
2014-04-02 23:53 PDT, Jer Noble
no flags
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion (557.41 KB, application/zip)
2014-04-03 00:42 PDT, Build Bot
no flags
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 (553.82 KB, application/zip)
2014-04-03 00:49 PDT, Build Bot
no flags
Patch (6.97 KB, patch)
2014-04-03 00:58 PDT, Jer Noble
no flags
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2 (609.59 KB, application/zip)
2014-04-03 01:21 PDT, Build Bot
no flags
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion (552.63 KB, application/zip)
2014-04-03 05:15 PDT, Build Bot
no flags
Archive of layout-test-results from webkit-ews-01 for mac-mountainlion (550.42 KB, application/zip)
2014-04-03 06:18 PDT, Build Bot
no flags
Jer Noble
Comment 1 2014-04-02 23:48:02 PDT
Jer Noble
Comment 2 2014-04-02 23:53:07 PDT
Build Bot
Comment 3 2014-04-03 00:42:16 PDT
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
Build Bot
Comment 4 2014-04-03 00:42:19 PDT
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
Build Bot
Comment 5 2014-04-03 00:49:16 PDT
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
Build Bot
Comment 6 2014-04-03 00:49:18 PDT
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
Jer Noble
Comment 7 2014-04-03 00:58:58 PDT
Build Bot
Comment 8 2014-04-03 01:21:15 PDT
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
Build Bot
Comment 9 2014-04-03 01:21:19 PDT
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
Build Bot
Comment 10 2014-04-03 05:15:33 PDT
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
Build Bot
Comment 11 2014-04-03 05:15:36 PDT
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
Build Bot
Comment 12 2014-04-03 06:18:36 PDT
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
Build Bot
Comment 13 2014-04-03 06:18:39 PDT
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
Jer Noble
Comment 14 2014-04-03 08:12:42 PDT
So far, the fixes for YouTube's behavior have broken more than they fix :-(
Jer Noble
Comment 15 2014-04-03 08:14:57 PDT
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
Jer Noble
Comment 16 2014-04-03 22:43:09 PDT
Nope, the access log is a AVPlayerItem construct, so it won't exist at the point where YouTube is making its seek request.
Jer Noble
Comment 17 2014-04-03 22:44:13 PDT
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.
Csaba Osztrogonác
Comment 18 2014-06-04 00:53:03 PDT
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).
Note You need to log in before you can comment on or make changes to this bug.