936021 - JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 53: reference to undefined property this.treeBoxObject.view

Status: NEW

(Toolkit :: General, defect)

Description

[ISHIKAWA, Chiaki]

During the run of |make mozmill| test suite of comm-central thunderbird, I got the
warning of the following form 11 times.

JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 53: reference to undefined property this.treeBoxObject.view

A program should not access undefined property.

The following is the list of such undefined property errors encounted during the execution of |make mozmill|.

The number at the beginning of the line is the number of appearances of the
warning during |make mozmill| test suite run.

     55 JavaScript strict warning: chrome://messenger/content/tabmail.xml, line 352: reference to undefined property aTabType.panelId
     55 JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 50: reference to undefined property this.treeBoxObject.columns
     55 JavaScript strict warning: chrome://global/content/bindings/findbar.xml, line 262: reference to undefined property this._browser.finder
     35 JavaScript error: chrome://global/content/bindings/findbar.xml, line 262: this._browser.finder is undefined
     12 JavaScript strict warning: chrome://messenger/content/newmailaccount/jquery.tmpl.js, line 123: reference to undefined property def[1]
***  11 JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 53: reference to undefined property this.treeBoxObject.view
      3 JavaScript strict warning: chrome://messenger/content/nsContextMenu.js, line 102: reference to undefined property this.onEditableArea
      1 JavaScript strict warning: resource:///modules/dbViewWrapper.js, line 1046: reference to undefined property this._underlyingFolders[0]

The code in question on line 53 look like this, and 
I chose DOM apps because I saw "nsIDOMXULTreeElement" in the comment line slight above it.

--- quote
    <implementation implements="nsIDOMXULTreeElement, nsIDOMXULMultiSelectControlElement">

      <!-- ///////////////// nsIDOMXULTreeElement ///////////////// -->

50=>  <property name="columns"
                onget="return this.treeBoxObject.columns;"/>

53=>   <property name="view"
                onget="return this.treeBoxObject.view;"
                onset="return this.treeBoxObject.view = val;"/>
--- end quote

Given that we have a top-most error that shows up 55 times related on line 50,

    55 JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 50: reference to undefined property this.treeBoxObject.columns

I suspect that there may not be a careful handling of this data structure, especially when an error occurs, etc.

TIA

PS: The first occurrences are preceded by errors as follows.


WARNING: NS_ENSURE_SUCCESS(res, res) failed with result 0x80004005: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/html/nsHTMLEditRules.cpp, line 316
WARNING: NS_ENSURE_TRUE(sheet) failed: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/html/nsHTMLEditor.cpp, line 2959
WARNING: NS_ENSURE_TRUE(aNode) failed: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/base/nsEditor.cpp, line 3479
JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 53: reference to undefined property this.treeBoxObject.view
Step Pass: {"function": "Controller.type()"}

     ... in another test

WARNING: NS_ENSURE_SUCCESS(res, res) failed with result 0x80004005: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/text/nsTextEditRules.cpp, line 417
WARNING: NS_ENSURE_TRUE(aNode) failed: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/base/nsEditor.cpp, line 3479
JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 53: reference to undefined property this.treeBoxObject.view
Step Pass: {"function": "Controller.keypress()"}

     ...

WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/base/nsEditor.cpp, line 3784
WARNING: NS_ENSURE_SUCCESS(res, res) failed with result 0x80004005: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/text/nsTextEditRules.cpp, line 417
WARNING: NS_ENSURE_TRUE(aNode) failed: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/base/nsEditor.cpp, line 3479
JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 53: reference to undefined property this.treeBoxObject.view

	   ..

WARNING: NS_ENSURE_TRUE(sheet) failed: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/html/nsHTMLEditor.cpp, line 2959
WARNING: NS_ENSURE_TRUE(aNode) failed: file /REF-COMM-CENTRAL/comm-central/mozilla/editor/libeditor/base/nsEditor.cpp, line 3479
++DOMWINDOW == 175 (0x654d328) [serial = 550] [outer = 0xc62b58]
WARNING: NS_ENSURE_TRUE(enabled) failed: file /REF-COMM-CENTRAL/comm-central/mozilla/dom/base/Navigator.cpp, line 1707
JavaScript strict warning: chrome://global/content/bindings/tree.xml, line 53: reference to undefined property this.treeBoxObject.view
Step Pass: {"function": "Controller.type()"}
Step Pass: {"function": "Controller.type()"}

...

Comment 1

[ISHIKAWA, Chiaki]

I am reassigning this to core:xbl
TIA