Summary: | REGRESSION (r280374): ASSERTION FAILED: Completion handler should not be called more than once under WebCore::MediaControlsContextMenuProvider::contextMenuItemSelected | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> | ||||||||
Component: | Media | Assignee: | Devin Rousso <hi> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | calvaris, eric.carlson, ews-watchlist, glenn, hi, jer.noble, philipj, sergio, webkit-bot-watchers-bugzilla, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | Other | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=228310 | ||||||||||
Attachments: |
|
Description
Ryan Haddad
2021-08-02 17:03:05 PDT
It looks like this started with https://trac.webkit.org/changeset/280374/webkit Created attachment 434946 [details]
Patch
Created attachment 434950 [details]
Patch
oops `MediaControlsContextMenuProvider::didDismissContextMenu` isn't supposed to `std::exchange`, as that's kinda the whole point of this patch 😅
Committed r280676 (240278@main): <https://commits.webkit.org/240278@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 434950 [details]. |