Closed
Bug 208539
Opened 21 years ago
Closed 21 years ago
extend notifyOfInsertion api to include the sourceDocument
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
Tracking
()
RESOLVED
FIXED
People
(Reporter: Brade, Assigned: Brade)
Details
(Keywords: topembed)
Attachments
(1 file)
17.53 KB,
patch
|
Brade
:
review+
kinmoz
:
superreview+
|
Details | Diff | Splinter Review |
nsIContentFilter's notifyOfInsertion needs to have the source dom document.
Assignee | ||
Comment 1•21 years ago
|
||
Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 125115 [details] [diff] [review] patch to add source doc param r=rogermw in another bug
Attachment #125115 -
Flags: superreview?(sfraser)
Attachment #125115 -
Flags: review+
Comment on attachment 125115 [details] [diff] [review] patch to add source doc param sr=kin@netscape.com ==== If the args to this call take |const nsString&| it might be more efficient to just declare an |nsString emptyStr;| once and pass that as the arg instead of using |nsString()| ... not required for checkin of the patch, but we should probably change it if we can at some point. return InsertHTMLWithCharsetAndContext(aInString, nsString(), nsString(), - nsString(), nsString(), nsnull, 0, PR_TRUE); + nsString(), nsString(), + nsnull, nsnull, 0, PR_TRUE);
Attachment #125115 -
Flags: superreview?(sfraser) → superreview+
Assignee | ||
Comment 4•21 years ago
|
||
fix checked in to trunk
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•