Bug 92718 - Move createAudioDevice from platform to WebFrameClient
Summary: Move createAudioDevice from platform to WebFrameClient
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sailesh Agrawal
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-31 00:21 PDT by Sailesh Agrawal
Modified: 2012-08-01 14:29 PDT (History)
9 users (show)

See Also:

Attachments
Patch (9.39 KB, patch)
2012-07-31 00:46 PDT, Sailesh Agrawal 		no flags Details | Formatted Diff | Diff
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 Sailesh Agrawal 2012-07-31 00:21:34 PDT 
Move createAudioDevice from platform to WebFrameClient
Comment 1 Sailesh Agrawal 2012-07-31 00:46:55 PDT 
Created attachment 155458 [details]
Patch
Comment 2 WebKit Review Bot 2012-07-31 00:49:50 PDT 
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment 3 Adam Barth 2012-08-01 10:58:55 PDT 
This patch isn not marked for review.  Please let me know if you'd like me to review it.
Comment 4 Sailesh Agrawal 2012-08-01 11:05:17 PDT 
(In reply to comment #3)
> This patch isn not marked for review.  Please let me know if you'd like me to review it.

There's still some discussion on https://chromiumcodereview.appspot.com/10836025/ about the my approach for this feature. Once croggers and tommi are ok with it I'll send this out for review. Thanks!
Comment 5 Chris Rogers 2012-08-01 14:29:47 PDT 
Based on our discussion, it sounds like we'll need to at least iron out the layering violation issue with AudioContext here...