UserMediaPermissionRequestManager should not remove ActivityStateChangeObserver in destructor
<rdar://problem/44196724>
Created attachment 349072 [details] Patch
Comment on attachment 349072 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=349072&action=review > Source/WebKit/WebProcess/WebPage/WebPage.cpp:710 > +#endif I believe this cleanup call might be too late. How about doing it in WebPage::close() instead since it is the place where m_page is gone? If done there, setting m_userMediaPermissionRequestManager to nullptr in WebPage::close() might be good enough.
Created attachment 349406 [details] Updated patch
Comment on attachment 349406 [details] Updated patch Clearing flags on attachment: 349406 Committed r235918: <https://trac.webkit.org/changeset/235918>
All reviewed patches have been landed. Closing bug.