Closed Bug 522769 Opened 14 years ago Closed 14 years ago
XUL tree widget can call tree view set
Cell Text multiple times
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:18.104.22.168) Gecko/2009082706 Firefox/3.0.14 Build Identifier: Songbird 1.5.0a build 1321 In the XUL tree widget stopEditing method, stopEditing may be reentered via the call to setCellText. Since this._editingColumn is not cleared until after stopEditing is called, stopEditing can be called a second time. In the case of the Songbird filter pane widget's use of the XUL tree widget, setCellText sends a request to a thread and waits for the result, processing the event queue in the meantime. Another UI event is on the queue that causes stopEditing to be called again. If the text change causes a change in the filter results, the tree rows will change and the second call to setCellText will target the incorrect item. However, it shouldn't be called a second time. Reproducible: Sometimes Steps to Reproduce: STR 1) Launch Songbird 1.5 build 1321. 2) Import music from multiple artists. 3) Select one of the artists in the artist filter pane. 4) Change the artist name in one of the tracks. Actual Results: Artist name changed in selected track plus an additional track from different artist. Expected Results: Artist name changed in only one track. See http://bugzilla.songbirdnest.com/show_bug.cgi?id=18399
Comment on attachment 406746 [details] [diff] [review] Changed the XUL tree widget to clear _editingColumn before calling setCellText in stopEditing. Per Mook's suggestion.
Attachment #406746 - Flags: review?(enndeakin)
Comment on attachment 406746 [details] [diff] [review] Changed the XUL tree widget to clear _editingColumn before calling setCellText in stopEditing. Songbird is looking to migrate to the 1.9.2 branch from 1.9.0 and would like to not have to maintain this patch locally
This can already land on trunk however.
Status: UNCONFIRMED → RESOLVED
Closed: 14 years ago
Component: XUL → XUL Widgets
Product: Core → Toolkit
QA Contact: xptoolkit.widgets → xul.widgets
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
Comment on attachment 406746 [details] [diff] [review] Changed the XUL tree widget to clear _editingColumn before calling setCellText in stopEditing. approval1.9.2 requests aren't currently being monitored, since we're nearing RC freeze and there are too many outstanding requests, so I'm clearing this request. Feel free to re-request approval if you are confident that it's worth drivers' time to consider whether this non-blocker needs to land for 1.9.2 at this stage.
You need to log in before you can comment on or make changes to this bug.