Assumed that getting a TreeModel iterator would always work isn't true
at shudown when the model is being depopulated.  This removes
assertion and only proceeds when valid every time.