Closed Bug 907883 Opened 6 years ago Closed 6 years ago

Minimize #includes in layout/generic

Categories

(Core :: Layout, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: ehsan, Assigned: ehsan)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.
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!
Attachment #793633 - Flags: review?(roc) → review+
(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.  :-)
https://hg.mozilla.org/mozilla-central/rev/f10d31a0f7b4
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.