Bug 1678062 - Remove unnecessary includes. r=andi

Differential Revision: https://phabricator.services.mozilla.com/D97467

Depends on D96561

Author: sigiesec

accessible/aom/AccessibleNode.h

@@ -10,7 +10,6 @@
 #include "nsDataHashtable.h"
 #include "nsRefPtrHashtable.h"
 #include "nsWrapperCache.h"
-#include "mozilla/ErrorResult.h"
 #include "mozilla/dom/BindingDeclarations.h"
 #include "mozilla/dom/DOMString.h"
 #include "mozilla/dom/Nullable.h"
@@ -19,6 +18,8 @@ class nsINode;
 
 namespace mozilla {
 
+class ErrorResult;
+
 namespace a11y {
 class Accessible;
 }

devtools/shared/heapsnapshot/DominatorTree.h

@@ -8,12 +8,13 @@
 
 #include "mozilla/devtools/HeapSnapshot.h"
 #include "mozilla/dom/BindingDeclarations.h"
-#include "mozilla/ErrorResult.h"
 #include "mozilla/RefCounted.h"
 #include "js/UbiNodeDominatorTree.h"
 #include "nsWrapperCache.h"
 
 namespace mozilla {
+class ErrorResult;
+
 namespace devtools {
 
 class DominatorTree final : public nsISupports, public nsWrapperCache {

devtools/shared/heapsnapshot/HeapSnapshot.h

@@ -7,7 +7,6 @@
 #define mozilla_devtools_HeapSnapshot__
 
 #include "js/HashTable.h"
-#include "mozilla/ErrorResult.h"
 #include "mozilla/devtools/DeserializedNode.h"
 #include "mozilla/dom/BindingDeclarations.h"
 #include "mozilla/dom/Nullable.h"
@@ -27,6 +26,8 @@
 #include "nsXPCOM.h"
 
 namespace mozilla {
+class ErrorResult;
+
 namespace devtools {
 
 class DominatorTree;

docshell/base/BrowsingContext.h

@@ -15,7 +15,6 @@
 #include "mozilla/RefPtr.h"
 #include "mozilla/Span.h"
 #include "mozilla/Tuple.h"
-#include "mozilla/WeakPtr.h"
 #include "mozilla/dom/BindingDeclarations.h"
 #include "mozilla/dom/FeaturePolicy.h"
 #include "mozilla/dom/LocationBase.h"

docshell/base/SyncedContext.h

@@ -7,12 +7,16 @@
 #ifndef mozilla_dom_SyncedContext_h
 #define mozilla_dom_SyncedContext_h
 
+#include <array>
+#include <type_traits>
+#include <utility>
+#include "mozilla/Attributes.h"
 #include "mozilla/EnumSet.h"
+#include "nsStringFwd.h"
+#include "nscore.h"
+
+// Referenced via macro definitions
 #include "mozilla/ErrorResult.h"
-#include "mozilla/Maybe.h"
-#include "mozilla/RefPtr.h"
-#include "mozilla/Tuple.h"
-#include <utility>
 
 class PickleIterator;
 
@@ -31,6 +35,8 @@ struct IPDLParamTraits;
 namespace dom {
 class ContentParent;
 class ContentChild;
+template <typename T>
+class MaybeDiscarded;
 
 namespace syncedcontext {
 

docshell/base/WindowContext.h

@@ -9,7 +9,6 @@
 
 #include "mozilla/PermissionDelegateHandler.h"
 #include "mozilla/Span.h"
-#include "mozilla/dom/BrowsingContext.h"
 #include "mozilla/dom/MaybeDiscarded.h"
 #include "mozilla/dom/SyncedContext.h"
 #include "mozilla/dom/UserActivation.h"
@@ -27,6 +26,7 @@ namespace dom {
 
 class WindowGlobalParent;
 class WindowGlobalInit;
+class BrowsingContext;
 class BrowsingContextGroup;
 
 #define MOZ_EACH_WC_FIELD(FIELD)                                       \

docshell/base/nsILoadContext.idl

@@ -14,7 +14,9 @@ webidl Element;
 
 %{C++
 #ifdef MOZILLA_INTERNAL_API
-#include "mozilla/BasePrincipal.h" // for OriginAttributes
+namespace mozilla {
+class OriginAttributes;
+}
 #endif
 %}
 

docshell/shistory/ChildSHistory.h

@@ -21,9 +21,9 @@
 
 #include "nsCOMPtr.h"
 #include "mozilla/dom/BindingDeclarations.h"
-#include "mozilla/ErrorResult.h"
 #include "nsWrapperCache.h"
 #include "nsThreadUtils.h"
+#include "mozilla/ErrorResult.h"
 #include "mozilla/LinkedList.h"
 #include "nsID.h"
 

dom/abort/AbortController.h

@@ -8,14 +8,15 @@
 #define mozilla_dom_AbortController_h
 
 #include "mozilla/dom/BindingDeclarations.h"
-#include "mozilla/ErrorResult.h"
 #include "nsCOMPtr.h"
 #include "nsCycleCollectionParticipant.h"
 #include "nsWrapperCache.h"
 
 class nsIGlobalObject;
 
 namespace mozilla {
+class ErrorResult;
+
 namespace dom {
 
 class AbortSignal;

dom/animation/CSSPseudoElement.h

@@ -9,7 +9,6 @@
 
 #include "js/TypeDecls.h"
 #include "mozilla/Attributes.h"
-#include "mozilla/ErrorResult.h"
 #include "mozilla/dom/BindingDeclarations.h"
 #include "mozilla/RefPtr.h"
 #include "nsCSSPseudoElements.h"