453277 - Investigate if we can avoid nulling out the tree in nsAbView::ClearView

Status: NEW

(MailNews Core :: Address Book, defect)

Description

[Mark Banner (:standard8)]

This is filed as a result of discussions/investigations on bug 451257.

nsAbView::ClearView currently clears the stored tree and selection, we then have to set it back up again in abResultPane.js/setAbView. It seems more logical that we set up the tree and selection once until the view is destroyed.

What needs to be checked is that the performance doesn't regress when switching views, because that was the reason for clearing the tree in the first place (although that was probably the wrong way to do it).

Comment 1

[Wayne Mery (:wsmwk)]

a

Comment 2

[Alessandro Castellani [:aleca]]

I've no idea about this, Geoff will know better.