Bug 53125 - [GTK] LayoutTests/media/audio-mpeg4-supported.html fails
Summary: [GTK] LayoutTests/media/audio-mpeg4-supported.html fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL: http://build.webkit.org/results/GTK%2...
Keywords:
Depends on: 53230
Blocks:
  Show dependency tree / graph
 
Reported: 2011-01-25 14:10 PST by Jer Noble
Modified: 2011-02-03 12:05 PST (History)
7 users (show)

See Also:

Attachments
proposed patch (3.30 KB, patch)
2011-01-26 05:34 PST, Philippe Normand 		no flags Details | Formatted Diff | Diff
updated patch (4.12 KB, patch)
2011-01-27 03:48 PST, Philippe Normand 		mrobinson: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jer Noble 2011-01-25 14:10:59 PST 
It has been suggested that the cause of the failure is that the necessary (non free) codecs are not installed on the build bots.

Test added in: http://trac.webkit.org/changeset/76621
Original bug: https://bugs.webkit.org/show_bug.cgi?id=52467
Comment 1 Jer Noble 2011-01-25 14:23:52 PST 
Test added to LayoutTests/platform/gtk/Skipped in: https://trac.webkit.org/changeset/76640
Comment 2 Philippe Normand 2011-01-26 05:34:21 PST 
Created attachment 80188 [details]
proposed patch
Comment 3 Philippe Normand 2011-01-26 05:37:44 PST 
The codecs are installed on the buildbots (gst-ffmpeg), the issue is just that the GStreamer player didn't handle the audio/x-m4a mimetype correctly.
Comment 4 Martin Robinson 2011-01-26 09:18:11 PST 
Comment on attachment 80188 [details]
proposed patch

Thanks for the quick fix!
Comment 5 Philippe Normand 2011-01-27 00:56:13 PST 
Committed r76773: <http://trac.webkit.org/changeset/76773>
Comment 6 Philippe Normand 2011-01-27 02:01:54 PST 
Will send a new patch as the first one broke some tests, sorry :(
Comment 7 Philippe Normand 2011-01-27 03:48:23 PST 
Created attachment 80315 [details]
updated patch
Comment 8 Philippe Normand 2011-02-03 06:17:06 PST 
Ping? Martin? :)
Comment 9 Philippe Normand 2011-02-03 08:30:48 PST 
Committed r77494: <http://trac.webkit.org/changeset/77494>
Comment 10 WebKit Review Bot 2011-02-03 12:05:58 PST 
http://trac.webkit.org/changeset/77494 might have broken GTK Linux 32-bit Debug