Some `nsIEditor.insertNode` callers set unknown 4th arguments
Categories
(Thunderbird :: Message Compose Window, defect, P3)
Tracking
(thunderbird_esr102 wontfix)
Tracking | Status | |
---|---|---|
thunderbird_esr102 | --- | wontfix |
People
(Reporter: masayuki, Assigned: masayuki)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Starting from the initial release of nsIEditor.insertNode
, it has only 3 arguments, but some callers set true
or false
. According to the comments under suite/editor/components
, it's intended to prevent editor to update Selection
.
Currently, I'm try to add such argument in bug 1764895, but I don't like to use bool
because of unclear from the caller sites. Therefore, first, I'd like to correct current code in comm-central first.
Assignee | ||
Comment 1•2 years ago
|
||
Oddly, the API never has 4th argument. However, some callers set it to true
to prevent the editor updating Selection
. Some of them also use
nsIEditor.setShouldTxnSetSelection
correctly, but the others do not.
This patch just removes them with adding XXX comment only where the wrong
callers to keep current behavior.
Assignee | ||
Comment 2•2 years ago
|
||
frg: I don't find your account in Phabricator. Could you check attachment 9304653 [details]?
Comment 3•2 years ago
|
||
We don't use phab in suite. Dones not suit us :) I am ok with the patch. If you need and r+ from suite I added IanN.
Interesting. Nothing to find about the fourth parameter even going to cvs. So it is safe to remove. I will take it to our current release branch and check there too. Central builds but is still broken.
Thanks and r+ for the suite parts and stated.
Assignee | ||
Comment 4•2 years ago
|
||
Thank you, frg. It's fine to land it without the stamp because it's important that somebody who maintains around the changes checks it.
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/comm-central/rev/87360d9e8dca
Make the callers of nsIEditor.insertNode
stop using unknown 4th argument r=mkmelin
Updated•2 years ago
|
Description
•