RESOLVED FIXED 115894
Remove [NoInterfaceObject] from several WebAudio IDL interfaces 
https://bugs.webkit.org/show_bug.cgi?id=115894
Summary Remove [NoInterfaceObject] from several WebAudio IDL interfaces
Chris Dumez
Reported 2013-05-10 02:16:20 PDT
A lot of WebAudio IDL interfaces have the [NoInterfaceObject] extended attribute, meaning that there is no corresponding attribute on the global window object. This is not according to specification [1] and should probably be fixed. [1] https://dvcs.w3.org/hg/audio/raw-file/tip/webaudio/specification.html
Attachments
WIP Patch (24.80 KB, patch)
2013-05-10 02:18 PDT, Chris Dumez 		buildbot: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from webkit-ews-05 for mac-mountainlion (529.75 KB, application/zip)
2013-05-10 13:40 PDT, Build Bot 		no flags
Details
Patch (43.46 KB, patch)
2013-05-10 17:17 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2013-05-10 02:18:57 PDT
Created attachment 201326 [details] WIP Patch
Chris Rogers
Comment 2 2013-05-10 11:56:05 PDT
Kentaro, I assume you will review this one?
Build Bot
Comment 3 2013-05-10 13:40:35 PDT
Comment on attachment 201326 [details] WIP Patch Attachment 201326 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/448136 New failing tests: fast/js/global-constructors-attributes.html
Build Bot
Comment 4 2013-05-10 13:40:38 PDT
Created attachment 201417 [details] Archive of layout-test-results from webkit-ews-05 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-05 Port: mac-mountainlion Platform: Mac OS X 10.8.2
Chris Dumez
Comment 5 2013-05-10 17:17:11 PDT
Created attachment 201443 [details] Patch
WebKit Commit Bot
Comment 6 2013-05-10 18:24:10 PDT
Comment on attachment 201443 [details] Patch Clearing flags on attachment: 201443 Committed r149920: <http://trac.webkit.org/changeset/149920>
WebKit Commit Bot
Comment 7 2013-05-10 18:24:15 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.