Bug 135751

Summary: [WebAudio] FFTFrame is cluttered with dead code
Product: WebKit Reporter: Philippe Normand <pnormand>
Component: Web AudioAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, crogers, eric.carlson, glenn, jer.noble, philipj, pnormand, sergio
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch eric.carlson: review+

Description Philippe Normand 2014-08-08 00:55:54 PDT
There are currently 4 FFTFrame implementations but I think that only 2 are used, the one for Mac and the GStreamer implementation.

FFTFrame.h has also some ifdefs WEBAUDIO_OPENMAX_DL_FFT which were specific to Chromium, I think we should remove this.

Thoughts?
Comment 1 Eric Carlson 2014-08-08 08:43:15 PDT
(In reply to comment #0)
> There are currently 4 FFTFrame implementations but I think that only 2 are used, the one for Mac and the GStreamer implementation.
> 
> FFTFrame.h has also some ifdefs WEBAUDIO_OPENMAX_DL_FFT which were specific to Chromium, I think we should remove this.
> 
> Thoughts?

Out with the old and unused!
Comment 2 Philippe Normand 2014-08-08 09:08:58 PDT
Ok, I'm too busy now but I'll prepare a patch next week.
Comment 3 Philippe Normand 2014-08-11 00:34:49 PDT
Created attachment 236354 [details]
Patch
Comment 4 Philippe Normand 2014-08-11 02:13:03 PDT
Created attachment 236359 [details]
Patch

Also removing unused ifdefs from AudioArray.h
Comment 5 Eric Carlson 2014-08-11 06:52:39 PDT
Comment on attachment 236359 [details]
Patch

Thanks Philippe!
Comment 6 Philippe Normand 2014-08-12 00:50:09 PDT
Committed r172431: <http://trac.webkit.org/changeset/172431>