288706 - InitAndRestoreFrame isn't always called

Status: NEW

(Core :: Layout, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

From bzbarsky bug 3247 comment 87:
> Also, there are a few places in the frame constructor that don't call
> InitAndRestoreFrame but may want to affect counters (the replacement wrapper
> frame for <img> elements and letter frames, to be precise)...

From dbaron in bug 3247 comment 88:
> Not calling InitAndRestoreFrame is a bug, and fixing those bugs shouldn't be in
> the scope of this patch.

Comment 1

[Boris Zbarsky [:bzbarsky]]

So what places _should_ call InitAndRestoreFrame?  I only mentioned the ones
that I thought would be relevant to counters... We also have the various frames
we construct for CSS anonymous content that manually call Init(), the various
special frames like the canvas and what not, continuation frames created via the
frame constructor, etc, etc.