674055 - can't collapse+expand accounts in folder pane, tree is undefined

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect)

Description

[David :Bienvenu]

+++ This bug was initially created as a clone of Bug #658534 +++

tbird trunk circa 2011-05

expected: account folders expanded

result: Error: uncaught exception: [Exception... "'[JavaScript Error: "tree is undefined" {file: "chrome://messenger/content/folderPane.js" line: 889}]' when calling method: [nsITreeView::toggleOpenState]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: chrome://global/content/bindings/tree.xml :: changeOpenState :: line 241"  data: yes]

As you can see in the patch for bug 658534, explicitly passing in "this" to our nested function fixes the issue, but my understanding is that the original code should have kept working. Apologies if that's not the case.

Comment 1

[Chris Coulson]

We have some users in Ubuntu hit this too, and they confirm the observation in https://bugzilla.mozilla.org/show_bug.cgi?id=658534#c13 regarding the problem going away after deleting the startup cache. Running thunderbird with -purgecaches seems to prevent the problem from occurring entirely

Comment 2

[Wolfgang Rosenauer [:wolfiR]]

I have the same issue with 6.0b1. This makes TB6 highly unusable for me currently.

Comment 3

[David Mandelin [:dmandelin]]

Wolfgang, does purging the caches fix this for you?

Comment 4

[Wolfgang Rosenauer [:wolfiR]]

Hmm, sorry, I had 6.0b2 already installed which has the mentioned workaround.

Comment 5

[David :Bienvenu]

(In reply to Wolfgang Rosenauer [:wolfiR] from comment #4)
> Hmm, sorry, I had 6.0b2 already installed which has the mentioned workaround.

Does this mean you can't reproduce the issue with b2?

Comment 6

[Wolfgang Rosenauer [:wolfiR]]

It hasn't happened for me since 6.0b2 and that is how I understood bug 658534

Comment 7

[David :Bienvenu]

(In reply to Wolfgang Rosenauer [:wolfiR] from comment #6)
> It hasn't happened for me since 6.0b2 and that is how I understood bug 658534

great, thanks, just making sure.