Closed
Bug 661292
Opened 14 years ago
Closed 14 years ago
Make nsChangeObserver::ContentInserted not use the index parameter
Categories
(Core :: Widget: Cocoa, defect)
Core
Widget: Cocoa
Tracking
()
RESOLVED
FIXED
mozilla7
People
(Reporter: mounir, Assigned: mounir)
References
Details
Attachments
(1 file, 1 obsolete file)
|
6.69 KB,
patch
|
jaas
:
review+
|
Details | Diff | Splinter Review |
Unfortunately, I *have* to use IndexOf() given that, if I understand it correctly, we have an array of menu that reflects the DOM and we will have to look for the index at some point. It will be slower but nothing significant I believe.
Attachment #536657 -
Flags: review?(joshmoz)
|
Assignee | |
Comment 1•14 years ago
|
||
I did update the wrong patch... :(
Attachment #536657 -
Attachment is obsolete: true
Attachment #536657 -
Flags: review?(joshmoz)
Attachment #536666 -
Flags: review?(joshmoz)
Attachment #536666 -
Flags: review?(joshmoz) → review+
|
|
Assignee | |
Updated•14 years ago
|
Flags: in-testsuite-
Whiteboard: [needs review] → [fixed in cedar]
|
|
Assignee | |
Comment 2•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in cedar]
Target Milestone: --- → mozilla7
You need to log in
before you can comment on or make changes to this bug.
Description
•