On Friday 2003-10-10 03:01 -0500, Boris Zbarsky wrote:
> Other than causing false positives on the layout regression tests, does 
> the NS_FRAME_HAS_LOADED_IMAGES have a reason for existing?  A quick lxr 
> search gives 
> (<http://lxr.mozilla.org/mozilla/search?string=NS_FRAME_HAS_LOADED_IMAGES>):
> 
> /layout/base/public/nsIFrame.h, line 172 -- #define 
> NS_FRAME_HAS_LOADED_IMAGES 0x00000080
> /layout/base/src/nsPresContext.cpp, line 1420 -- 
> aTargetFrame->AddStateBits(NS_FRAME_HAS_LOADED_IMAGES);
> 
> and no other hits.
> 
> It seems to me like we could remove this flag (and have a free 
> framestate bit, yay!).

http://bugzilla.mozilla.org/show_bug.cgi?id=201076 covers the false
positives on the regression test.

http://bugzilla.mozilla.org/show_bug.cgi?id=97934 covers another frame
state bit that could be removed.  I wouldn't be surprised if there are
others as well.

-David

-- 
L. David Baron                                <URL: http://dbaron.org/ >
_______________________________________________
Mozilla-layout mailing list
[EMAIL PROTECTED]
http://mail.mozilla.org/listinfo/mozilla-layout

Reply via email to