Bug 919871: Make .cpp files in layout/generic #include their corresponding .h file first. r=mats

layout/generic/nsBlockFrame.cpp

@@ -9,11 +9,12 @@
  * boxes, also used for various anonymous boxes
  */
 
+#include "nsBlockFrame.h"
+
 #include "mozilla/DebugOnly.h"
 #include "mozilla/Util.h"
 
 #include "nsCOMPtr.h"
-#include "nsBlockFrame.h"
 #include "nsAbsoluteContainingBlock.h"
 #include "nsBlockReflowContext.h"
 #include "nsBlockReflowState.h"

layout/generic/nsBlockReflowState.cpp

@@ -6,6 +6,8 @@
 
 /* state used in reflow of block frames */
 
+#include "nsBlockReflowState.h"
+
 #include "mozilla/DebugOnly.h"
 
 #include "nsBlockFrame.h"

layout/generic/nsBulletFrame.cpp

@@ -5,8 +5,9 @@
 
 /* rendering object for list-item bullets */
 
-#include "nsCOMPtr.h"
 #include "nsBulletFrame.h"
+
+#include "nsCOMPtr.h"
 #include "nsGkAtoms.h"
 #include "nsGenericHTMLElement.h"
 #include "nsAttrValueInlines.h"

layout/generic/nsFrame.cpp

@@ -6,11 +6,12 @@
 
 /* base class of all rendering objects */
 
+#include "nsFrame.h"
+
 #include "mozilla/Attributes.h"
 #include "mozilla/DebugOnly.h"
 
 #include "nsCOMPtr.h"
-#include "nsFrame.h"
 #include "nsFrameList.h"
 #include "nsPlaceholderFrame.h"
 #include "nsIContent.h"

layout/generic/nsFrameSetFrame.cpp

@@ -5,10 +5,11 @@
 
 /* rendering object for HTML <frameset> elements */
 
+#include "nsFrameSetFrame.h"
+
 #include "mozilla/DebugOnly.h"
 #include "mozilla/Likely.h"
 
-#include "nsFrameSetFrame.h"
 #include "nsGenericHTMLElement.h"
 #include "nsAttrValueInlines.h"
 #include "nsLeafFrame.h"

layout/generic/nsGfxScrollFrame.cpp

@@ -5,13 +5,14 @@
 
 /* rendering object to wrap rendering objects that should be scrollable */
 
+#include "nsGfxScrollFrame.h"
+
 #include "base/compiler_specific.h"
 #include "nsCOMPtr.h"
 #include "nsPresContext.h"
 #include "nsView.h"
 #include "nsIScrollable.h"
 #include "nsContainerFrame.h"
-#include "nsGfxScrollFrame.h"
 #include "nsGkAtoms.h"
 #include "nsINameSpaceManager.h"
 #include "nsContentList.h"

layout/generic/nsHTMLCanvasFrame.cpp

@@ -5,8 +5,9 @@
 
 /* rendering object for the HTML <canvas> element */
 
-#include "nsGkAtoms.h"
 #include "nsHTMLCanvasFrame.h"
+
+#include "nsGkAtoms.h"
 #include "mozilla/dom/HTMLCanvasElement.h"
 #include "nsDisplayList.h"
 #include "nsLayoutUtils.h"

layout/generic/nsHTMLReflowState.cpp

@@ -5,6 +5,8 @@
 
 /* struct containing the input to nsIFrame::Reflow */
 
+#include "nsHTMLReflowState.h"
+
 #include "nsStyleConsts.h"
 #include "nsCSSAnonBoxes.h"
 #include "nsFrame.h"

layout/generic/nsImageFrame.cpp

@@ -5,10 +5,11 @@
 
 /* rendering object for replaced elements with bitmap image data */
 
+#include "nsImageFrame.h"
+
 #include "mozilla/DebugOnly.h"
 
 #include "nsCOMPtr.h"
-#include "nsImageFrame.h"
 #include "nsIImageLoadingContent.h"
 #include "nsString.h"
 #include "nsPrintfCString.h"

layout/generic/nsLineLayout.cpp

@@ -5,11 +5,12 @@
 
 /* state and methods used while laying out a single line of a block frame */
 
+// This has to be defined before nsLineLayout.h is included, because
+// nsLineLayout.h has a #include for plarena.h, which needs this defined:
 #define PL_ARENA_CONST_ALIGN_MASK (sizeof(void*)-1)
-#include "plarena.h"
+#include "nsLineLayout.h"
 
 #include "mozilla/Util.h"
-#include "nsLineLayout.h"
 #include "nsBlockFrame.h"
 #include "nsStyleConsts.h"
 #include "nsContainerFrame.h"

layout/generic/nsObjectFrame.cpp

@@ -6,6 +6,8 @@
 
 /* rendering objects for replaced elements implemented by a plugin */
 
+#include "nsObjectFrame.h"
+
 #ifdef XP_WIN
 // This is needed for DoublePassRenderingEvent.
 #include "mozilla/plugins/PluginMessageUtils.h"
@@ -36,7 +38,6 @@
 #include "GeckoProfiler.h"
 #include <algorithm>
 
-#include "nsObjectFrame.h"
 #include "nsIObjectFrame.h"
 #include "nsPluginNativeWindow.h"
 #include "FrameLayerBuilder.h"

layout/generic/nsSimplePageSequence.cpp

@@ -2,8 +2,10 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-#include "nsCOMPtr.h"
+
 #include "nsSimplePageSequence.h"
+
+#include "nsCOMPtr.h"
 #include "nsPresContext.h"
 #include "gfxContext.h"
 #include "nsRenderingContext.h"

layout/generic/nsSubDocumentFrame.cpp

@@ -8,9 +8,10 @@
  * as <frame>, <iframe>, and some <object>s
  */
 
+#include "nsSubDocumentFrame.h"
+
 #include "mozilla/layout/RenderFrameParent.h"
 
-#include "nsSubDocumentFrame.h"
 #include "nsCOMPtr.h"
 #include "nsGenericHTMLElement.h"
 #include "nsAttrValueInlines.h"

layout/generic/nsVideoFrame.cpp

@@ -6,10 +6,11 @@
 
 /* rendering object for the HTML <video> element */
 
+#include "nsVideoFrame.h"
+
 #include "nsCOMPtr.h"
 #include "nsGkAtoms.h"
 
-#include "nsVideoFrame.h"
 #include "mozilla/dom/HTMLVideoElement.h"
 #include "nsIDOMHTMLVideoElement.h"
 #include "nsIDOMHTMLImageElement.h"