Closed Bug 1055894 Opened 10 years ago Closed 1 year ago

nsIFrame should provide 'normal' version of the logical coordinate getters

Categories

(Core :: Layout, defect)

defect

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: seth, Assigned: TYLin)

References

Details

Attachments

(1 file)

nsFrame exposes GetRect() and GetPosition(), which report information about the frame's size and location. It also exposes GetNormalRect() and GetNormalPosition(), which report the same information as it would be if relative positioning had not been applied. This is used extensively in some parts of the layout code.

We've recently added new logical coordinate getters such as BStart() and GetLogicalPosition(). It would be very useful if we added "normal" variations of these which called GetNormalRect() and GetNormalPosition() under the hood.
I've added a dependency on bug 35168 because the modifications to nsTableFrame::GetLogicalBaseline() added in part 2 are very awkward, and would be made much simpler if we had "normal" logical coordinate getters. We should update that code when we add the getters.
(Actually this depends on bug 35168 anyway, since it adds GetNormalRect(), which we'll probably want to use in this bug.)
Severity: normal → S3

We've already had GetLogicalNormalPosition().

This patch makes the API nicer, and shouldn't change the behavior.

Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Pushed by tlin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7437637d0b5c
Add GetLogicalNormalRect() and adapt some callers of GetNormalRect(). r=emilio
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
You need to log in before you can comment on or make changes to this bug.