Bug 96644 - Issue with convolver node usage in webkit2:webAudio API
Summary: Issue with convolver node usage in webkit2:webAudio API
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Audio (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P1 Blocker
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-13 07:17 PDT by kdj
Modified: 2023-03-23 08:38 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description kdj 2012-09-13 07:17:44 PDT
Hi,

I was trying to run few WebAudio API applications from chrome site; especially for convolverNode.

App name: Digital DJ 
Link: http://chromium.googlecode.com/svn/trunk/samples/audio/dj.html
This application did not work properly, for small fraction the audio came and further the silence played out.

After removing the convolverNode; referring the lines in application.
...
line no.: 386: //convolver.connect(preCompressorGain);
...
line no.: 397: //setReverbImpulseResponse('impulse-responses/filter-rhythm2.wav');
...

The application worked perfectly :-) 

To investigate further, with enabled convolverNode and I added data dump logs in following places at same time:
1. ConvolverNode::process() for input(0)->bus(), to check input to convolverNode.
2. webKitWebAudioSrcLoop(), for (const_cast<float*>(priv->bus->channel(index)->data())).

The data for 1. is complete silence whereas for 2. it is proper convoluted stream.

Here it seems that the WebAudio API convolverNode has issue before connecting to other node. 
I tried to other application which are using ConvolverNode; result is same "for small fraction the audio came and further the silence played out".

Please share your views.

Thanks and regards,
kaustubh
Comment 1 Philippe Normand 2012-09-13 07:36:15 PDT
That demo works fine here with your patch from bug 95833. Without that patch I can hear the pop noise glitches of bug 82347 (so I'll close the last as dupe of the first btw).

Tested with a 64-bit release build of webkitgtk.

Should we close this one?
Comment 2 kdj 2012-09-13 19:34:34 PDT
Good it is working on 64-bit !

I have tested with 32-bit machine.

Could you please share your 64-bit release details on which it is working. So that I can try for the same one at my end.

Is it OK to take tar ball "WebKit r128500" ?

Thanks,
kdj
Comment 3 Philippe Normand 2012-09-13 23:21:02 PDT
What kind of details about my setup do you need?
Comment 4 kdj 2012-09-14 03:44:04 PDT
I am getting this error with WebKit r128500:

  CXX    Source/JavaScriptCore/Programs_jsc_3-jsc.o
  CXX    Source/WebKit2/gtk/Programs_WebKitWebProcess-MainGtk.o
  CXXLD  Programs/WebKitPluginProcess
libtool: link: cannot find the library `libjavascriptcoregtk-3.0.la' or unhandled argument `libjavascriptcoregtk-3.0.la'
make[1]: *** [Programs/WebKitPluginProcess] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory `/home/newgen/webkit-org/WebKit-r128500/WebKitBuild/Release'
make: *** [all] Error 2

I tried with this path:  http://savannah.gnu.org/bugs/?30653

Could you please tell/guide the exact patch for the same ?
Comment 5 Philippe Normand 2012-09-14 03:52:51 PDT
If you do build-webkit --gtk --update-gtk it should work. Please join #webkitgtk+ if you have build issues, bugzilla is not the best place to discuss about that.
Comment 6 kdj 2012-09-15 04:24:51 PDT
Ok, I registered to #webkitgtk+ and able to build on 64bit webkitGtk with webaudio.
I tried to run the same app link. But not able to hear the audio. 

Please tell me anything more you are doing at your end.
Comment 7 Philippe Normand 2012-09-16 22:51:39 PDT
Have you set the AUDIO_RESOURCES_PATH env variable to point to .../Source/WebCore/platform/audio/resources ?
Comment 8 kdj 2012-09-17 00:29:34 PDT
yes, after setting same Env var, Convolver node is working; thanks philippe.