Closed Bug 862180 Opened 7 years ago Closed 7 years ago

Make nsDisplayItem::GetUnderlyingFrame infallible

Categories

(Core :: Layout, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: roc, Assigned: roc)

Details

Attachments

(2 files)

And rename it to just Frame().
Until recently, nsDisplayClip items could have a null mFrame, but that doesn't happen anymore.
Attachment #737791 - Flags: review?(matt.woodrow) → review+
Comment on attachment 737792 [details] [diff] [review]
Part 2: Rename nsDisplayItem::GetUnderlyingFrame() to Frame()

Review of attachment 737792 [details] [diff] [review]:
-----------------------------------------------------------------

This is going to take me a while to adjust to!
Attachment #737792 - Flags: review?(matt.woodrow) → review+
https://hg.mozilla.org/mozilla-central/rev/4675831aff1c
https://hg.mozilla.org/mozilla-central/rev/67ceea995837
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.