856312 - Make EditorBase into a skippable class

Status: NEW

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

Description

[Andrew McCreight [:mccr8]]

This isn't a big deal, but right now the start of the traverse for nsEditor sometimes returns NS_SUCCESS_INTERRUPTED_TRAVERSE.  We could move that logic into CanSkip, remove that from traverse, and turn it into a NS_IMPL_CYCLE_COLLECTION_n class.

Comment 1

[(no longer active)]

I'm not sure what any of this means!  :-)

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

A skippable object is something which can be removed from the cycle collector's purple buffer if we
can detect that it is certainly alive.

Comment 3

[Andrew McCreight [:mccr8]]

Right now, there's some logic at the start of the TRAVERSE method for nsEditor that detects if the editor is definitely alive.  If it is, then the CC doesn't need to look at it any more.

The "old" way to make use of this information is to return NS_SUCCESS_INTERRUPTED_TRAVERSE from the TRAVERSE method, but the more modern way is to do that reasoning in various CanSkip methods, which are at various earlier points.

Comment 4

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.