350332 - Editable trees don't work in chrome windows

Status: VERIFIED FIXED

(Core :: XUL, defect)

Description

[Nickolay_Ponomarev]

STR:
1. Open the example from http://wiki.mozilla.org/XUL:Tree#Usage (quoted below) as a chrome document.
2. Double click on the "[e-mail]" cell.

Actual results:
No textbox appears to edit the contents of the cell. (I believe it appears and instantly disappears.) The tree is not focused (or at least the selected row is painted with the selected-not-focused color).

Expected results:
Textbox appears in place of the cell.

It is weird, but when the document is opened without chrome privileges, inline editing basically works.

Tested with trunk build pulled sometime yesterday.

Comment 1

[Nickolay_Ponomarev]

Attachment: testcase

<?xml version="1.0"?>
<?xml-stylesheet href="chrome://global/skin/" type="text/css"?>
<window id="yourwindow" xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 <tree editable="true" flex="1">
   <treecols>
     <treecol id="sender" label="Sender" flex="1"/>
     <treecol id="subject" label="Subject" flex="2"/>
   </treecols>
   <treechildren>
     <treeitem>
       <treerow>
         <treecell label="[e-mail]"/>
         <treecell label="Top secret plans" editable="false"/>
       </treerow>
     </treeitem>
   </treechildren>
 </tree>
</window>

Comment 2

[Nickolay_Ponomarev]

This appears to be caused by the same reason as bug 350350 - http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/toolkit/content/widgets/tree.xml&rev=1.37&mark=408#400 handling text selection, in this case, initial.

Comment 3

[Nickolay_Ponomarev]

*** Bug 350350 has been marked as a duplicate of this bug. ***

Comment 4

[Nickolay_Ponomarev]

Verified with a Windows build I just made. Yay, thanks.