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 Bugs

Assignee:

Jer Noble <jer.noble>

Status:

RESOLVED WONTFIX

Severity:

Normal

CC:

buildbot, commit-queue, eric.carlson, glenn, philipj, rniwa, sergio

Priority:

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	Details Formatted Diff Diff
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	Details
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	Details
Patch (6.97 KB, patch) 2014-04-03 00:58 PDT, Jer Noble	no flags	Details Formatted Diff Diff
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	Details
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	Details
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	Details
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

Jer Noble

Comment 1 2014-04-02 23:48:02 PDT

<rdar://problem/16382745>

Jer Noble

Comment 2 2014-04-02 23:53:07 PDT

Created attachment 228474 [details] Patch

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

Created attachment 228486 [details] Patch

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.