Bug 44212 - text/plain non-video files cause <video> to hang (while reading them?)
Summary: text/plain non-video files cause <video> to hang (while reading them?)
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Jer Noble
URL: http://hixie.ch/tests/adhoc/html/vide...
Keywords: InRadar
Depends on:
Reported: 2010-08-18 16:14 PDT by Ian 'Hixie' Hickson
Modified: 2010-08-30 17:11 PDT (History)
5 users (show)

See Also:

Disable Text Components (15.45 KB, patch)
2010-08-27 17:02 PDT, Jer Noble
eric.carlson: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ian 'Hixie' Hickson 2010-08-18 16:14:18 PDT
1. Load http://hixie.ch/tests/adhoc/html/video/002.html
2. Immediately interact with the page.

Can't do step 2.

No hanging.

It works fine for shorter text/plain files (http://hixie.ch/tests/adhoc/html/video/003.html) so my guess is that it's blocking while downloading the whole file.
Comment 1 Eric Carlson 2010-08-19 08:10:11 PDT
Comment 2 Jer Noble 2010-08-27 17:02:17 PDT
Created attachment 65790 [details]
Disable Text Components
Comment 3 Eric Seidel (no email) 2010-08-27 17:21:10 PDT
Attachment 65790 [details] did not build on mac:
Build output: http://queues.webkit.org/results/3877032
Comment 4 Eric Carlson 2010-08-27 17:37:40 PDT
Comment on attachment 65790 [details]
Disable Text Components

These changes look OK, but you forgot to regenerate the WebKitSystemInterface libraries.

r=me with the missing libraries.
Comment 5 Jer Noble 2010-08-30 14:47:22 PDT
Committed r66411: <http://trac.webkit.org/changeset/66411>
Comment 6 Jer Noble 2010-08-30 15:26:50 PDT
Committed r66415:  http://trac.webkit.org/changeset/66415
Comment 7 Ryosuke Niwa 2010-08-30 17:11:27 PDT
http://trac.webkit.org/changeset/66418 seems to have caused 20+ files to fail on WebKit2:

Tests that timed out:
compositing/layers-inside-overflow-scroll.html	stderr
compositing/self-painting-layers2.html	stderr
compositing/self-painting-layers.html	stderr
compositing/geometry/clipped-video-controller.html	stderr
compositing/geometry/video-fixed-scrolling.html	stderr
compositing/geometry/video-opacity-overlay.html	stderr
compositing/overflow/overflow-compositing-descendant.html	stderr
compositing/overflow/scroll-ancestor-update.html	stderr
compositing/reflections/load-video-in-reflection.html	stderr
compositing/video/video-background-color.html	stderr
fast/dom/beforeload/remove-video-in-beforeload-listener.html	stderr
fast/js/cyclic-proto.html	stderr
fast/selectors/001.html	stderr
media/audio-constructor-preload.html	stderr
media/audio-constructor-src.html	stderr
media/audio-constructor.html	stderr
media/audio-controls-rendering.html	stderr
media/audio-data-url.html	stderr
media/audio-mpeg-supported.html	stderr
media/audio-only-video-intrinsic-size.html	stderr

My apologies if they are just flaky but I'm letting you know just in case.