Remove no-longer-used argument from nsBlockFrame::DrainOverflowLines(nsBlockReflowState& aState)

RESOLVED FIXED in mozilla12

Status

()

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

Trunk
mozilla12
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

8 years ago
The method
   nsBlockFrame::DrainOverflowLines(nsBlockReflowState& aState)
no longer uses its argument aState.

These csets removed the last usages, as part of bug 492627:
http://hg.mozilla.org/mozilla-central/diff/28008648ca04/layout/generic/nsBlockFrame.cpp#l1.185
http://hg.mozilla.org/mozilla-central/diff/e9130436ada8/layout/generic/nsBlockFrame.cpp#l1.436

It looks like we can just take DrainOverflowLines() argumentless now.
Assignee

Comment 1

8 years ago
Posted patch fix v1Splinter Review
I think this'll do it. (haven't finished rebuilding yet though)
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Assignee

Updated

8 years ago
Attachment #592876 - Flags: review?(roc)
https://hg.mozilla.org/mozilla-central/rev/94bae0abad47
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.