654998 - Annoying error console messages with JS-driven TreeViews: reference to undefined property X.view

Status: NEW

(Thunderbird :: Mail Window Front End, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

Mozilla/5.0 (X11; Linux i686; rv:6.0a1) Gecko/20110504 Thunderbird/3.4a1pre

Steps to reproduce:

1)  Open up Thunderbird's Error Console
2)  Click on any folder in the 3pane folder tree

What happens?

Some errors pop up in the console, saying:  

Warning: reference to undefined property b.view
Source File: chrome://global/content/bindings/tree.xml
Line: 1022

This same bug appears to surface in the address book with the patch in bug 422845.  In that case, the error is:

Warning: reference to undefined property this.treeBoxObject.view
Source File: chrome://global/content/bindings/tree.xml
Line: 0

It seems the Javascript-driven TreeViews might be prone to this error.

Comment 1

[Ludovic Hirlimann [:Usul]]

Mike shouldn't this be in XUL or is it us miss using the component ?

Comment 2

[Mike Conley (:mconley) (:⚙️)]

I'm unsure at this point - but I have a feeling we may have skipped a step or two in setting up our Javascript TreeViews.  That's just instinct talking, though.

Comment 3

[:Irving Reid (No longer working on Firefox)]

This is related to Bug 383524 and Bug 646307; some C++ code is calling QueryInterface on a Javascript object that doesn't implement QI. There are a bunch of similar bugs, but nothing that looks like a real attempt to fix QI for JavaScript objects.

Comment 4

[Philipp Wagner [:imphil]]

Irving: thanks, adding a 

QueryInterface: XPCOMUtils.generateQI([Components.interfaces.nsITreeView])

to my tree implementation fixes this! Probably the mailnews error is the same.

Comment 5

[:aceman]

Can anybody still see this? I think I saw it ocassionally, but the STRs are probably not that easy today as originally in comment 0.