Closed Bug 1799407 Opened 3 months ago Closed 3 months ago

Move the definitions in FrameChildList.h/cpp into nsFrameList.h/cpp

Categories

(Core :: Layout, task)

task

Tracking

()

RESOLVED FIXED
108 Branch
Tracking Status
firefox108 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

Details

Attachments

(1 file)

We have comments [1] [2] saying that putting definitions in other h/cpp solves circular dependencies, but we really can just write them in nsFrameFrame.h/cpp for simplicity.

[1] https://searchfox.org/mozilla-central/rev/3c194fa1d6f339036d2ec9516bd310c6ad612859/layout/generic/nsFrameList.h#332
[2] https://searchfox.org/mozilla-central/rev/3c194fa1d6f339036d2ec9516bd310c6ad612859/layout/generic/FrameChildList.h#19

There shouldn't be any circular dependency that requires putting definitions in
FrameChildList.h/cpp.

Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/c5d97a474aeb
Move definitions in FrameChildList.h/cpp into nsFrameList.h/cpp. r=emilio
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch
You need to log in before you can comment on or make changes to this bug.