Otherwise, the assertions might do some use after free errors.
Created attachment 330680 [details] Patch
<rdar://problem/36347765>
<rdar://problem/36347764>
Comment on attachment 330680 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=330680&action=review > Source/WebKit/UIProcess/WebProcessPool.cpp:1461 > +#ifndef NDEBUG This looks suspicious. Why do we only need to protect |this| in debug?
Comment on attachment 330680 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=330680&action=review >> Source/WebKit/UIProcess/WebProcessPool.cpp:1461 >> +#ifndef NDEBUG > > This looks suspicious. Why do we only need to protect |this| in debug? We only have ASSERT() after the call to requestTermination which may cause WebProcessPool to be freed.
Comment on attachment 330680 [details] Patch Clearing flags on attachment: 330680 Committed r226519: <https://trac.webkit.org/changeset/226519>
All reviewed patches have been landed. Closing bug.