Closed
Bug 234960
Opened 21 years ago
Closed 21 years ago
nsIDocumentObserver::ContentChanged is silly
Categories
(Core :: DOM: Core & HTML, defect, P1)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla1.7beta
People
(Reporter: dbaron, Assigned: dbaron)
Details
(Whiteboard: [patch])
Attachments
(1 file)
|
74.96 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
nsIDocumentObserver::ContentChanged and thus nsIFrame::ContentChanged is used
only for changes to character data (although a bunch of frames assume it could
be used for other things and proceed to handle it in a way they consider "logical").
I have a patch to:
* change the name to CharacterDataChanged
* change the nsISupports parameter to a boolean parameter, since we don't need
to allocate nsTextContentChangedData objects to pass a boolean.
|
Assignee | |
Updated•21 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [patch]
Target Milestone: --- → mozilla1.7beta
|
|
Assignee | |
Comment 1•21 years ago
|
||
|
|
Assignee | |
Updated•21 years ago
|
Attachment #141780 -
Flags: superreview?(jst)
Attachment #141780 -
Flags: review?(jst)
|
||
Comment 2•21 years ago
|
||
Comment on attachment 141780 [details] [diff] [review]
patch
Nice! :-)
r+sr=jst
Attachment #141780 -
Flags: superreview?(jst)
Attachment #141780 -
Flags: superreview+
Attachment #141780 -
Flags: review?(jst)
Attachment #141780 -
Flags: review+
|
|
Assignee | |
Comment 3•21 years ago
|
||
Fix checked in to trunk, 2004-02-20 13:37/38 -0800.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•