120190 - Make sure nsOutlinerContentView::AttributeChanged() doesn't die in recursive loop

Status: VERIFIED DUPLICATE of bug 120189

(Core :: XUL, defect)

Description

[Jan Varga [:janv]]

<bryner> Jan: um, so, i think removing mIgnoreOptionSelected will cause problems
<Jan> bryner: I thought that there are used 2 different attributes
<bryner> oh, right
<bryner> nevermind
<bryner> :)
<bryner> er
<bryner> no, i think it will cause a problem
<bryner> here's why
<bryner> if an option is selected via script, the optionSelectedPseudo attribute
gets set
<bryner> that causes us to call into the outliner selection to toggle the
selected state...
<bryner> which then calls back into the content view to update state on the
SelectElement
<bryner> which then resets the optionSelectedPseudo attribute
<bryner> and around and around we go
<Jan> bryner: I'll file a new bug on me to investigate this case, I can't test
it again with all outliners :(
<bryner> it will only affect xbl select widgets
<Jan> yeah I know
<bryner> Jan: i sort of need those to work :)
<bryner> this would be solved by moving the select element update into the xbl
event handlers like one of your patches did
<bryner> it seems like there was a problem with that, and i can't remember what
it was
<Jan> yes!
<Jan> move it, move it

so maybe we just eliminate all callback notifications in C++
by moving them to XBL

Comment 1

[John Morrison]

entered twice.

*** This bug has been marked as a duplicate of 120189 ***

Comment 2

[Jan Varga [:janv]]

sorry, my connection just died when I was filling this bug :(

Comment 3

[Christopher Aillon (sabbatical, not receiving bugmail)]

v