907883 - Minimize #includes in layout/generic

Status: RESOLVED FIXED

(Core :: Layout, defect)

Comment 1

[(no longer active)]

Attachment: Patch (v1)

https://tbpl.mozilla.org/?tree=Try&rev=10d2330a31f2

Comment 2

[(no longer active)]

https://tbpl.mozilla.org/?tree=Try&rev=ca9d08dce565

Comment 3

[Mats Palmgren (inactive)]

Comment on attachment 793633 [details] [diff] [review]
Patch (v1)

> layout/base/nsPresContext.cpp
+#include "nsObjectFrame.h"

nsObjectFrame.h is already included (~17 lines above).

Looks good otherwise. r=mats for the version that compiles ;-)

Nice!

Comment 4

[(no longer active)]

https://tbpl.mozilla.org/?tree=Try&rev=fd2c014211df

Comment 5

[(no longer active)]

(In reply to Mats Palmgren (:mats) from comment #3)
> Comment on attachment 793633 [details] [diff] [review]
> Patch (v1)
> 
> > layout/base/nsPresContext.cpp
> +#include "nsObjectFrame.h"
> 
> nsObjectFrame.h is already included (~17 lines above).

Oops you're right!  No idea why I did that.  :-)

Comment 6

[(no longer active)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/f10d31a0f7b4

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/f10d31a0f7b4