297991 - clean up document viewer ownership/destruction

Status: NEW

(Core :: Layout, defect)

Description

[Brian Ryner (not reading)]

Currently, document viewer destruction is somewhat fragile -- it's likely to
assert in certain cases even though things are "ok", and in general we don't do
a good job of calling Destroy() before the last reference is released.  We need to:

a) figure out whether Destroy() is really needed -- does it break some cycle?
apparently not in all cases, if we can enter the destructor without Destroy
being called.
b) clean up the refcounting games that we play to make the printing code happy