Bug 1673931 - Avoid including ErrorResult.h in generated bindings when not needed. r=nika

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

dom/bindings/BindingDeclarations.h

@@ -34,6 +34,11 @@ class nsIPrincipal;
 class nsWrapperCache;
 
 namespace mozilla {
+
+class ErrorResult;
+class OOMReporter;
+class CopyableErrorResult;
+
 namespace dom {
 
 class BindingCallContext;

dom/bindings/Codegen.py

@@ -17990,7 +17990,6 @@ class CGBindingRoot(CGThing):
             (
                 "mozilla/dom/BindingDeclarations.h",
                 "mozilla/dom/Nullable.h",
-                "mozilla/ErrorResult.h",
             ),
             True,
         )
@@ -18237,6 +18236,9 @@ class CGBindingRoot(CGThing):
             hasNonEmptyDictionaries or jsImplemented or callbackDescriptors
         )
 
+        if callbackDescriptors:
+            bindingDeclareHeaders["mozilla/ErrorResult.h"] = True
+
         def descriptorClearsPropsInSlots(descriptor):
             if not descriptor.wrapperCache:
                 return False
@@ -23125,7 +23127,6 @@ class CGEventRoot(CGThing):
             [
                 config.getDescriptor(parent).headerFile,
                 "mozilla/Attributes.h",
-                "mozilla/ErrorResult.h",
                 "mozilla/dom/%sBinding.h" % interfaceName,
                 "mozilla/dom/BindingUtils.h",
             ],