a follow up from bug 862863. The need of the patch was to make document logging correct at shutdown stage. It get rids addref/release works what I find nice while Trev has concerns. Pulling the patch here to not get it lost.

> stage. It get rids addref/release works what I find nice while Trev has if you just want to get rid of AddRef / Release that why not just .swap() the mDocumentNode to kfg?

Comment on attachment 775660 [details] [diff] [review] patch so the thing is crazy things can happen when you shutdown an accessible esp a document (that crazy atk stuff that fires sync events), so I'd really rather not have accessibles pint at stuff longer than they have to here. Actually there's another issue which is that when you remove the DocAccessible from cache of documents it could get deleted in which case you'll write null to freed memory.

ok, doing a swap kunfuDethGreep and moving it to where we null out mPresShell is ok?

(In reply to alexander :surkov from comment #3) > ok, doing a swap kunfuDethGreep and moving it to where we null out > mPresShell is ok? I kind of think nulling out mPresShell should maybe happen before the parent->RemoveChild() because of that weird atk stuff, but that's sort of a different issue so I guess its fine.