1826918 - Make array of content nodes initializers split from its users

Status: NEW

(Core :: DOM: Editor, task)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(away 4/27 - 5/6)]

In a lot of places, HTMLEditor does:

AutoTArray<OwningNonNull<nsIContent>, N> arrayOfContents;
{
  arrayOfContents.AppendElement(...);
}
for (const OwningNonNull<nsIContent>& content : arrayOfContents) {
  can_run_script(MOZ_KnownLive(content));
}

If the first block is big, it's unclear that what it does and it's hard to write the main handling code without MOZ_KnownLive. Therefore, the first block should be split from the method.