Open Bug 1714905 Opened 4 years ago Updated 4 years ago

Audit whether 0-origin EffectsInfo::mVisible is what we supposed or not and if it's supposed change it to nsSize

Categories

(Core :: Layout, task)

Product:
Core
Component:
Layout
Type:
task

Tracking

()

Project Flags:
Fission Milestone Future

People

(Reporter: hiro, Unassigned)

References

(Blocks 1 open bug)

Details

I did naively think that EffectsInfo::mVisible is representing the position relative to the parent content's root (or the root of the parent content's root). The comment says actually something like that. But with various testing, I've tested iframe's bordeer, iframe's margin, iframe's padding, and also iframe's root document's margin/padding, but none of them contributes mVisible rect's x/y values. If x/y values are not necessarry at all, we can just use nsSize for that.

Given that now it's working properly as far as we've been aware of, this is not urgent at all.

Blocks: rendering-fission
Fission Milestone: --- → Future
You need to log in before you can comment on or make changes to this bug.