16477 - nsEditor::InsertTextImpl() leaks

Status: VERIFIED INVALID

(Core :: DOM: Editor, defect, P2)

Description

[Robert John Churchill]

nsEditor::InsertTextImpl() calls CreateTxnForInsertText() [around line # 1608]
which returns an allocated "InsertTextTxn *" but never frees it. Please fix.

[Note: leak detected using beard's Boehm GC]

Comment 1

[Joe Francis]

it's not supposed to free it.  ownership tranfers to the transaction manager. if
you mean that it never gets freed, even at quit time, that would be a different
bug that is known and not related to this code.

invalid.

Comment 2

[sujay]

verified in 10/18 build.

Comment 3

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/cc327d9c39a2
[fenix] For https://github.com/mozilla-mobile/fenix/issues/16477: Update a11y info to collection item for both grid and list.
https://hg.mozilla.org/releases/mozilla-beta/rev/f64b9910cf19
[fenix] For https://github.com/mozilla-mobile/fenix/issues/16477: Migrate updateAccessibilityCollectionItemInfo to extension.
https://hg.mozilla.org/releases/mozilla-beta/rev/1f706e1a4450
[fenix] For https://github.com/mozilla-mobile/fenix/issues/16477: Switch to using an extension to update a11y collection info.