bug 1601912 - "use" CRLite fields in NSSCertDBTrustDomain to silence compiler warnings r=jcj

When cert_storage is disabled, the CRLite mode and telemetry fields don't get
used by NSSCertDBTrustDomain, so we get warnings-as-errors about unused fields.
This uses Unused to silence the warnings.

This also adds a missing #include to CertBlocklist.cpp.

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

--HG--
extra : moz-landing-system : lando

security/certverifier/NSSCertDBTrustDomain.cpp

@@ -698,6 +698,11 @@ Result NSSCertDBTrustDomain::CheckRevocation(
       }
     }
   }
+#else
+  // "use" these fields to stop the compiler from complaining when
+  // MOZ_NEW_CERT_STORAGE is disabled.
+  Unused << mCRLiteMode;
+  Unused << mCRLiteTelemetryInfo;
 #endif
 
   // Bug 991815: The BR allow OCSP for intermediates to be up to one year old.

security/manager/ssl/CertBlocklist.cpp

@@ -14,6 +14,7 @@
 #include "mozilla/Preferences.h"
 #include "mozilla/Unused.h"
 #include "nsAppDirectoryServiceDefs.h"
+#include "nsComponentManagerUtils.h"  // for do_CreateInstance
 #include "nsDependentString.h"
 #include "nsDirectoryServiceUtils.h"
 #include "nsICryptoHash.h"