I have a question about this patch: In ~nsDocShell, Destroy() is already called before the added code and in nsDocShell::Destroy() the same cleanup code is already present (L5033). So wouldn't this cleanup already be done there?
Bug 1620818 Comment 20 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
I have a question about this patch: In ~nsDocShell, Destroy() is already called before the added code and in nsDocShell::Destroy() the same cleanup code is already present (L4521). So wouldn't this cleanup already be done there?