Closed Bug 458969 Opened 16 years ago Closed 16 years ago

dead stuff in nsSpaceManager

Categories

(Core :: Layout, defect)

Other
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: taras.mozilla, Assigned: taras.mozilla)

References

Details

Attachments

(1 file)

Attached patch dead stuffSplinter Review
took out mFrame even though it was being used in debug mode. Not sure if that's ok or not.
Attachment #342151 - Flags: review?(bzbarsky)
Except for the mFrame stuff, I'd guess this became unused when the reflow branch landed (bug 300030).
Comment on attachment 342151 [details] [diff] [review]
dead stuff

dbaron is probably a better reviewer here
Attachment #342151 - Flags: review?(bzbarsky) → review?(dbaron)
Comment on attachment 342151 [details] [diff] [review]
dead stuff

r+sr=dbaron if you merge the comment you're removing above GetFrame into the comment above the class definition by changing the comment above the class definition to:

Class for dealing with bands of available space. The space manager defines a coordinate space (relative to the frame that created the space manager) with an origin at (0, 0) that grows down and to the right.

(rewrapped at 72 characters)
Attachment #342151 - Flags: superreview+
Attachment #342151 - Flags: review?(dbaron)
Attachment #342151 - Flags: review+
pushed 4d78262505d7
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee: nobody → tglek
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: