Bug 225843

Summary: Clean up BaseAudioContext now that legacy/prefixed WebAudio is gone
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: Web AudioAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, cdumez, darin, eric.carlson, esprehn+autocc, ews-watchlist, ggaren, glenn, jer.noble, kondapallykalyan, peng.liu6, philipj, sergio, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=180332
https://bugs.webkit.org/show_bug.cgi?id=225849
Bug Depends on: 225832    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2021-05-15 13:14:26 PDT
Clean up BaseAudioContext now that legacy/prefixed WebAudio is gone. Now that we have a simpler class hierarchy, we can move more code out of BaseAudioContext and into its subclasses (OfflineAudioContext & AudioContext).
Attachments
Patch (73.80 KB, patch)
2021-05-15 14:57 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2021-05-15 14:57:07 PDT
Created attachment 428744 [details] Patch
Chris Dumez
Comment 2 2021-05-15 17:37:40 PDT
*** Bug 180332 has been marked as a duplicate of this bug. ***
EWS
Comment 3 2021-05-15 18:05:27 PDT
Committed r277553 (237781@main): <https://commits.webkit.org/237781@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 428744 [details].
Radar WebKit Bug Importer
Comment 4 2021-05-15 18:06:16 PDT
<rdar://problem/78066681>
Note You need to log in before you can comment on or make changes to this bug.