Bug 1652017 - Remove unnecessary includes from nsIFrame.h. r=emilio
authorSimon Giesecke <sgiesecke@mozilla.com>
Wed, 15 Jul 2020 13:48:35 +0000
changeset 540591 27cd47c0e5e66b3ca5985c9a2e3c3ffcf3366792
parent 540590 b473986ef1da9dc3b6489811c605b5656bd1dedf
child 540592 bf2bf7e462edb8491e2d7b31b0e6c8669cf091b3
push id121807
push usersgiesecke@mozilla.com
push dateWed, 15 Jul 2020 17:09:41 +0000
treeherderautoland@4916e2e9646c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio
bugs1652017
milestone80.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1652017 - Remove unnecessary includes from nsIFrame.h. r=emilio Differential Revision: https://phabricator.services.mozilla.com/D83110
layout/generic/BRFrame.cpp
layout/generic/nsIFrame.h
--- a/layout/generic/BRFrame.cpp
+++ b/layout/generic/BRFrame.cpp
@@ -7,16 +7,17 @@
 /* rendering object for HTML <br> elements */
 
 #include "mozilla/PresShell.h"
 #include "mozilla/dom/HTMLBRElement.h"
 #include "gfxContext.h"
 #include "nsCOMPtr.h"
 #include "nsContainerFrame.h"
 #include "nsFontMetrics.h"
+#include "nsHTMLParts.h"
 #include "nsIFrame.h"
 #include "nsPresContext.h"
 #include "nsLineLayout.h"
 #include "nsStyleConsts.h"
 #include "nsGkAtoms.h"
 #include "nsLayoutUtils.h"
 
 // FOR SELECTION
--- a/layout/generic/nsIFrame.h
+++ b/layout/generic/nsIFrame.h
@@ -58,21 +58,19 @@
 #include "mozilla/Result.h"
 #include "mozilla/SmallPointerArray.h"
 #include "mozilla/PresShell.h"
 #include "mozilla/WritingModes.h"
 #include "nsDirection.h"
 #include "nsFrameList.h"
 #include "nsFrameState.h"
 #include "mozilla/ReflowInput.h"
-#include "nsHTMLParts.h"
 #include "nsITheme.h"
 #include "nsLayoutUtils.h"
 #include "nsQueryFrame.h"
-#include "nsString.h"
 #include "mozilla/ComputedStyle.h"
 #include "nsStyleStruct.h"
 #include "Visibility.h"
 #include "nsChangeHint.h"
 #include "mozilla/ComputedStyleInlines.h"
 #include "mozilla/EnumSet.h"
 #include "mozilla/gfx/CompositorHitTestInfo.h"
 #include "mozilla/gfx/MatrixFwd.h"
@@ -96,72 +94,59 @@
  *    are re-positioned as well. It's reasonable to not position the view until
  *    after all reflowing the entire line, for example, but the frame should
  *    still be positioned and sized (and the view sized) during the reflow
  *    (i.e., before sending the DidReflow() notification)
  * 5. the view system handles moving of widgets, i.e., it's not our problem
  */
 
 class nsAtom;
-class nsPresContext;
 class nsView;
 class nsFrameSelection;
 class nsIWidget;
 class nsISelectionController;
 class nsBoxLayoutState;
 class nsBoxLayout;
 class nsILineIterator;
 class nsDisplayItemBase;
 class nsDisplayListBuilder;
 class nsDisplayListSet;
 class nsDisplayList;
 class gfxSkipChars;
 class gfxSkipCharsIterator;
 class gfxContext;
 class nsLineList_iterator;
 class nsAbsoluteContainingBlock;
-class nsIContent;
 class nsContainerFrame;
 class nsPlaceholderFrame;
 class nsStyleChangeList;
 class nsWindowSizes;
 
 struct nsBoxLayoutMetrics;
 struct nsPeekOffsetStruct;
-struct nsPoint;
-struct nsRect;
-struct nsSize;
-struct nsMargin;
 struct CharacterDataChangeInfo;
 
 namespace mozilla {
 
 enum class PseudoStyleType : uint8_t;
 enum class TableSelectionMode : uint32_t;
 class EventStates;
-class PresShell;
-struct ReflowInput;
-class ReflowOutput;
 class ServoRestyleState;
 class DisplayItemData;
 class EffectSet;
 
 namespace layers {
 class Layer;
 class LayerManager;
 }  // namespace layers
 
 namespace layout {
 class ScrollAnchorContainer;
 }  // namespace layout
 
-namespace dom {
-class Selection;
-}  // namespace dom
-
 }  // namespace mozilla
 
 //----------------------------------------------------------------------
 
 // 1 million CSS pixels less than our max app unit measure.
 // For reflowing with an "infinite" available inline space per [css-sizing].
 // (reflowing with an NS_UNCONSTRAINEDSIZE available inline size isn't allowed
 //  and leads to assertions)