Closed Bug 320337 Opened 19 years ago Closed 19 years ago

Remove mFrameConstructor member from nsListBoxBodyFrame

Categories

(Core :: Layout, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED

People

(Reporter: vhaarr+bmo, Assigned: vhaarr+bmo)

References

()

Details

Attachments

(1 file)

nsListBoxBodyFrame has a |nsCSSFrameConstructor* mFrameConstructor| member, it could just get it from |GetPresContext()->PresShell()->FrameConstructor()| when needed.
Attached patch version 1Splinter Review
Something like this?

I have to admit, I don't know what |class nsCSSFrameConstructor;| bit in nsListBoxBodyFrame.h does, I just noticed that it compiled fine when I removed it -- perhaps you can explain what that does, generally speaking?
Attachment #205930 - Flags: review?(roc)
"class nsCSSFrameConstructor;" basically says "nsCSSFrameConstructor is a class" and nothing more. You can then create pointers and references to it, but you can't access any fields or methods, or subclass from it, or call "new nsCSSFrameConstructor", etc etc without pulling in a real definition (from nsCSSFrameConstructor.h). xul/base/src/nsListBoxBodyFrame.h was using this so it could mention "nsCSSFrameConstructor*" in the places you removed (more efficient that doing a full #include "nsCSSFrameConstructor.h").
Ah, thank you.
roc: Could you help me with the checkin, too ? :-)
checked in. We should think about getting you CVS access sometime.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: