Commit f1b1f484 authored by philn@webkit.org's avatar philn@webkit.org

2011-03-15 Philippe Normand <pnormand@igalia.com>

        Reviewed by Eric Carlson.

        [GStreamer] http/tests/media/video-play-stall-before-meta-data.html fails
        https://bugs.webkit.org/show_bug.cgi?id=56370

        * platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
        (WebCore::mediaPlayerPrivateMessageCallback): Let the
        mediaPlayerClient handle the stream error, in this case the
        HTMLMediaElement will emit a stalled event.

2011-03-15  Philippe Normand  <pnormand@igalia.com>

        Reviewed by Eric Carlson.

        [GStreamer] http/tests/media/video-play-stall-before-meta-data.html fails
        https://bugs.webkit.org/show_bug.cgi?id=56370

        * platform/gtk/Skipped: Unskip fixed test.

git-svn-id: svn://svn.chromium.org/blink/trunk@81229 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 63163ca6
2011-03-15 Philippe Normand <pnormand@igalia.com>
Reviewed by Eric Carlson.
[GStreamer] http/tests/media/video-play-stall-before-meta-data.html fails
https://bugs.webkit.org/show_bug.cgi?id=56370
* platform/gtk/Skipped: Unskip fixed test.
2011-03-16 Yury Semikhatsky <yurys@chromium.org> 2011-03-16 Yury Semikhatsky <yurys@chromium.org>
Reviewed by Pavel Feldman. Reviewed by Pavel Feldman.
...@@ -498,9 +498,6 @@ http/tests/loading/gmail-assert-on-load.html ...@@ -498,9 +498,6 @@ http/tests/loading/gmail-assert-on-load.html
http/tests/loading/redirect-with-no-location-crash.html http/tests/loading/redirect-with-no-location-crash.html
http/tests/loading/text-content-type-with-binary-extension.html http/tests/loading/text-content-type-with-binary-extension.html
# https://bugs.webkit.org/show_bug.cgi?id=56370
http/tests/media/video-play-stall-before-meta-data.html
http/tests/media/video-play-stall-seek.html http/tests/media/video-play-stall-seek.html
http/tests/media/video-play-stall.html http/tests/media/video-play-stall.html
http/tests/media/video-seekable-stall.html http/tests/media/video-seekable-stall.html
......
2011-03-15 Philippe Normand <pnormand@igalia.com>
Reviewed by Eric Carlson.
[GStreamer] http/tests/media/video-play-stall-before-meta-data.html fails
https://bugs.webkit.org/show_bug.cgi?id=56370
* platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::mediaPlayerPrivateMessageCallback): Let the
mediaPlayerClient handle the stream error, in this case the
HTMLMediaElement will emit a stalled event.
2011-03-10 Philippe Normand <pnormand@igalia.com> 2011-03-10 Philippe Normand <pnormand@igalia.com>
Reviewed by Martin Robinson. Reviewed by Martin Robinson.
......
...@@ -117,6 +117,13 @@ gboolean mediaPlayerPrivateMessageCallback(GstBus* bus, GstMessage* message, gpo ...@@ -117,6 +117,13 @@ gboolean mediaPlayerPrivateMessageCallback(GstBus* bus, GstMessage* message, gpo
|| err->code == GST_RESOURCE_ERROR_NOT_FOUND) || err->code == GST_RESOURCE_ERROR_NOT_FOUND)
error = MediaPlayer::FormatError; error = MediaPlayer::FormatError;
else if (err->domain == GST_STREAM_ERROR) { else if (err->domain == GST_STREAM_ERROR) {
// Let the mediaPlayerClient handle the stream error, in
// this case the HTMLMediaElement will emit a stalled
// event.
if (err->code == GST_STREAM_ERROR_TYPE_NOT_FOUND) {
LOG_VERBOSE(Media, "Decode error, let the Media element emit a stalled event.");
break;
}
error = MediaPlayer::DecodeError; error = MediaPlayer::DecodeError;
attemptNextLocation = true; attemptNextLocation = true;
} else if (err->domain == GST_RESOURCE_ERROR) } else if (err->domain == GST_RESOURCE_ERROR)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment