Summary: | Evolution [GTK] Mail 3.10.4 and 3.12.4 can't read audio attachments from the integrated reader inline | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | meets | ||||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cgarcia, clopez, mcatanzaro, mcrha, pnormand | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
meets
2014-07-25 09:18:35 PDT
Hi Melodie, I just came back from holidays today, I'll try to check this bug tomorrow. Thanks for the report! I'm not sure to understand what's the issue exactly. The test case you attached is for WebKit1 which is no longer supported but anyway. :) What's supposed to happen when that test program is executed? Here the first audio element doesn't load because I don't have that wav file and the second audio element is a silence sound which plays fine here. Hi Philippe, Thank you for answering so fast. The test file has been provided to me by Milan Crha, from the Redhat development project. I uploaded a copy of the sound files so you can get them, here: http://meets.free.fr/evolution/ From the command lines used shown in the screen-shots in this directory: http://meets.free.fr/evolution/test-webkitgtk/ you might think the newer GTK3 version of WebKit has been used at some time during the tests. (Is that right?) The result expected from the test was to check weather the issue was coming from Evolution or rather from WebKitGTK. I am sending a mail to Milan Crha to show him your answer, and maybe could you talk together about this bug? This would seem more relevant to me, as I don't have coding skills. Best regards, Mélodie Focusing on WebKit2 because this is what we support from now on. I'm going to attach a wk2 port of test.c, changing local:// to file:// as base_uri works around the problem but I think that for a clean solution we need to reopen and land #130398. Created attachment 235688 [details]
wk2 port of test.c
(In reply to comment #3) > Hi Philippe, > > Thank you for answering so fast. > > The test file has been provided to me by Milan Crha, from the Redhat development project. I uploaded a copy of the sound files so you can get them, here: http://meets.free.fr/evolution/ > > From the command lines used shown in the screen-shots in this directory: http://meets.free.fr/evolution/test-webkitgtk/ you might think the newer GTK3 version of WebKit has been used at some time during the tests. (Is that right?) > No. Both WebKit1 and WebKit2 support GTK3. The biggest difference between WK1 and WK2 is that WK2 has a multi-process architecture. And of course their API is slightly different :) You can do a diff of the 2 versions of the test case to see how they differ. Hi, You just forgot to adjust the command line I need to start in the console (first line with the comments : "/* gcc test.c -g -O0 -o test `pkg-config --cflags --libs gtk+-3.0 webkitgtk-3.0` && ./test */"). If I try to adapt and use it with the right argument (your file name): =-=-=-=-=-=-= $ LANG=C gcc test-wk2.c -g -O0 -o test `pkg-config --cflags --libs gtk+-3.0 webkitgtk-3.0` && ./test /tmp/ccbZhRIw.o: In function `main': /tmp/test-wk2.c:23: undefined reference to `webkit_settings_new' /tmp/test-wk2.c:50: undefined reference to `webkit_web_view_load_html' collect2: error: ld returned 1 exit status =-=-=-=-=-=-= So you need to be very precise regarding instructions for testing because I don't know how to adapt it the right way. (In reply to comment #7) > Hi, > > You just forgot to adjust the command line I need to start in the console (first line with the comments : "/* gcc test.c -g -O0 -o test `pkg-config --cflags --libs gtk+-3.0 webkitgtk-3.0` && ./test */"). > > If I try to adapt and use it with the right argument (your file name): > > =-=-=-=-=-=-= > $ LANG=C gcc test-wk2.c -g -O0 -o test `pkg-config --cflags --libs gtk+-3.0 webkitgtk-3.0` && ./test > /tmp/ccbZhRIw.o: In function `main': > /tmp/test-wk2.c:23: undefined reference to `webkit_settings_new' > /tmp/test-wk2.c:50: undefined reference to `webkit_web_view_load_html' > collect2: error: ld returned 1 exit status > > =-=-=-=-=-=-= > > So you need to be very precise regarding instructions for testing because I don't know how to adapt it the right way. Try this: gcc test-wk2.c -g -O0 -o test `pkg-config --cflags --libs gtk+-3.0 webkit2gtk-3.0` And ensure you have the wekit2gtk development headers installed (On Debian testing/sid or Ubuntu trusty/utopic systems they are provided by the package libwebkit2gtk-3.0-dev) (In reply to comment #2) > What's supposed to happen when that test program is executed? Here the first audio element doesn't load because I don't have that wav file and the second audio element is a silence sound which plays fine here. The test works here (Fedora) the same as for you, the first file doesn't play (due to forbidden file:// URIs), the second plays fine, because it has assigned the data inline. This is how Evolution (an email client) does that, but it's not ported to webkit2 yet. The test.c is a minimal reproducer. The problem is that Melodie's system doesn't play any of the files, even it seems like he has the same packages installed. A better question would be, from my point of view, to know what he should have available locally to get the second file played, because it seems there's missing something. If you can give any pointers how to troubleshoot this, then it'll be great. (In reply to comment #9) > (In reply to comment #2) > > What's supposed to happen when that test program is executed? Here the first audio element doesn't load because I don't have that wav file and the second audio element is a silence sound which plays fine here. > > The test works here (Fedora) the same as for you, the first file doesn't play (due to forbidden file:// URIs), the second plays fine, because it has assigned the data inline. This is how Evolution (an email client) does that, but it's not ported to webkit2 yet. The test.c is a minimal reproducer. The problem is that Melodie's system doesn't play any of the files, even it seems like he has the same packages installed. > Wow, ok I was totally confused by the initial bug description then. Thanks for the clarification. > A better question would be, from my point of view, to know what he should have available locally to get the second file played, because it seems there's missing something. If you can give any pointers how to troubleshoot this, then it'll be great. To play that audio data: uri all you should need is gstreamer core, -plugins-base and -plugins-good. Can you please do this: GST_DEBUG_NO_COLOR=1 GST_DEBUG=5 ./test 2> gst.log and try to play the second audio element? Then attach the log on this bug, gzipped. Any news on this? (In reply to comment #11) > Any news on this? I haven't had news from this. Do I need to do something to help get through it? I have also been busy and lost sight about if I had a task to do to help, not sure about how Milan Crha and you have progressed. From comment #10: "To play that audio data: uri all you should need is gstreamer core, -plugins-base and -plugins-good. Can you please do this: GST_DEBUG_NO_COLOR=1 GST_DEBUG=5 ./test 2> gst.log and try to play the second audio element? Then attach the log on this bug, gzipped." Is this still an issue? Hello! Evolution 3.12.8 : now it works!! I don't know if it does in the former versions. I guess I'll have to try booting to Ubuntu (I do have a Unicorn/14.10 version). Thanks a lot for your work! Best regards, Mélodie Cool, I'm closing this bug then :) I am sending a mail to the original user who has shown this bug to me. Hopefully it works in his version too. (Ubuntu Trusty Tahr). Best regards, Mélodie |