Bug 73491

Summary: Enable web audio by default on non-Mac POSIX platforms
Product: WebKit Reporter: chromium
Component: Web AudioAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, crogers, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
enable web audio on unix none

Description chromium 2011-11-30 14:09:55 PST
enable web audio by default on non-Mac POSIX platforms
Comment 1 chromium 2011-11-30 14:26:07 PST
Build web audio by default on new Unix platforms, this patch has been used to build web audio on FreeBSD, Solaris, OpenBSD, and NetBSD.  I tested the Drum Machine web audio demo on FreeBSD and it appears to work fine.
Comment 2 chromium 2011-11-30 14:29:58 PST
Created attachment 117275 [details]
enable web audio on unix
Comment 3 chromium 2011-11-30 14:34:39 PST
Please review:

Chris - OK inclusion of other UNIX platforms for web audio
Adam  - Confirm Android exclusion
Tony  - Any additional input
Comment 4 Tony Chang 2011-11-30 14:41:03 PST
Seems fine to me.
Comment 5 Adam Barth 2011-11-30 14:47:44 PST
Excluding android is fine.
Comment 6 chromium 2011-12-06 13:15:13 PST
It looks like Chris is on vacation and won't get to this soon, but I'd like this patch to get into Chromium before the final 17.0 branch.  Perhaps Tony can commit this and Chris can revert later if he disagrees?  I doubt Chris will care about these other POSIX platforms since nothing changes for the main platforms with this patch, just thought I'd get his stamp since he's the web audio guy.
Comment 7 Tony Chang 2011-12-06 13:20:50 PST
Comment on attachment 117275 [details]
enable web audio on unix

I agree, this seems harmless.
Comment 8 WebKit Review Bot 2011-12-06 14:07:16 PST
Comment on attachment 117275 [details]
enable web audio on unix

Clearing flags on attachment: 117275

Committed r102178: <http://trac.webkit.org/changeset/102178>
Comment 9 WebKit Review Bot 2011-12-06 14:07:21 PST
All reviewed patches have been landed.  Closing bug.