208539 - extend notifyOfInsertion api to include the sourceDocument

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Description

[Kathleen :Brade]

nsIContentFilter's notifyOfInsertion needs to have the source dom document.

Comment 1

[Kathleen :Brade]

Attachment: patch to add source doc param

Comment 2

[Kathleen :Brade]

Comment on attachment 125115 [details] [diff] [review]
patch to add source doc param

r=rogermw in another bug

Comment 3

[kinmoz]

Comment on attachment 125115 [details] [diff] [review]
patch to add source doc param

sr=[e-mail]

==== 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);

Comment 4

[Kathleen :Brade]

fix checked in to trunk