nsListBoxBodyFrame has a |nsCSSFrameConstructor* mFrameConstructor| member, it could just get it from |GetPresContext()->PresShell()->FrameConstructor()| when needed.
Created attachment 205930 [details] [diff] [review] version 1 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?
"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
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.