[App Check] Remove usages of FIR_*_SUPPORTED_TARGETS macros (#11831)

Author: andrewheard

FirebaseAppCheck/Sources/AppAttestProvider/DCAppAttestService+FIRAppAttestService.h

@@ -14,17 +14,12 @@
  * limitations under the License.
  */
 
-#import <TargetConditionals.h>
-
-#import "FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRAppCheckAvailability.h"
-
-// Currently DCAppAttestService is available on iOS only.
-#if FIR_APP_ATTEST_SUPPORTED_TARGETS
-
 #import <DeviceCheck/DeviceCheck.h>
 
 #import "FirebaseAppCheck/Sources/AppAttestProvider/FIRAppAttestService.h"
 
+#import "FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRAppCheckAvailability.h"
+
 NS_ASSUME_NONNULL_BEGIN
 
 FIR_APP_ATTEST_PROVIDER_AVAILABILITY
@@ -33,5 +28,3 @@ FIR_APP_ATTEST_PROVIDER_AVAILABILITY
 @end
 
 NS_ASSUME_NONNULL_END
-
-#endif // FIR_APP_ATTEST_SUPPORTED_TARGETS

FirebaseAppCheck/Sources/AppAttestProvider/DCAppAttestService+FIRAppAttestService.m

@@ -16,11 +16,6 @@
 
 #import "FirebaseAppCheck/Sources/AppAttestProvider/DCAppAttestService+FIRAppAttestService.h"
 
-// Currently DCAppAttestService is available on iOS only.
-#if FIR_APP_ATTEST_SUPPORTED_TARGETS
-
 @implementation DCAppAttestService (FIRAppAttestService)
 
 @end
-
-#endif // FIR_APP_ATTEST_SUPPORTED_TARGETS

FirebaseAppCheck/Sources/AppAttestProvider/FIRAppAttestProvider.m

@@ -136,7 +136,6 @@ - (instancetype)initWithAppAttestService:(id<FIRAppAttestService>)appAttestServi
 }
 
 - (nullable instancetype)initWithApp:(FIRApp *)app {
-#if FIR_APP_ATTEST_SUPPORTED_TARGETS
  NSURLSession *URLSession = [NSURLSession
  sessionWithConfiguration:[NSURLSessionConfiguration ephemeralSessionConfiguration]];
 
@@ -166,9 +165,6 @@ - (nullable instancetype)initWithApp:(FIRApp *)app {
  keyIDStorage:keyIDStorage
  artifactStorage:artifactStorage
  backoffWrapper:backoffWrapper];
-#else // FIR_APP_ATTEST_SUPPORTED_TARGETS
- return nil;
-#endif // FIR_APP_ATTEST_SUPPORTED_TARGETS
 }
 
 #pragma mark - FIRAppCheckProvider

FirebaseAppCheck/Sources/DeviceCheckProvider/DCDevice+FIRDeviceCheckTokenGenerator.h

@@ -14,10 +14,6 @@
  * limitations under the License.
  */
 
-#import "FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRAppCheckAvailability.h"
-
-#if FIR_DEVICE_CHECK_SUPPORTED_TARGETS
-
 #import <DeviceCheck/DeviceCheck.h>
 
 #import "FirebaseAppCheck/Sources/DeviceCheckProvider/FIRDeviceCheckTokenGenerator.h"
@@ -29,5 +25,3 @@ NS_ASSUME_NONNULL_BEGIN
 @end
 
 NS_ASSUME_NONNULL_END
-
-#endif // FIR_DEVICE_CHECK_SUPPORTED_TARGETS

FirebaseAppCheck/Sources/DeviceCheckProvider/DCDevice+FIRDeviceCheckTokenGenerator.m

@@ -16,10 +16,6 @@
 
 #import "FirebaseAppCheck/Sources/DeviceCheckProvider/DCDevice+FIRDeviceCheckTokenGenerator.h"
 
-#if FIR_DEVICE_CHECK_SUPPORTED_TARGETS
-
 @implementation DCDevice (FIRDeviceCheckTokenGenerator)
 
 @end
-
-#endif // FIR_DEVICE_CHECK_SUPPORTED_TARGETS

FirebaseAppCheck/Sources/DeviceCheckProvider/FIRDeviceCheckProvider.m

@@ -14,10 +14,6 @@
  * limitations under the License.
  */
 
-#import "FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRAppCheckAvailability.h"
-
-#if FIR_DEVICE_CHECK_SUPPORTED_TARGETS
-
 #import <Foundation/Foundation.h>
 
 #if __has_include(<FBLPromises/FBLPromises.h>)
@@ -159,5 +155,3 @@ - (void)getTokenWithCompletion:(void (^)(FIRAppCheckToken *_Nullable token,
 @end
 
 NS_ASSUME_NONNULL_END
-
-#endif // FIR_DEVICE_CHECK_SUPPORTED_TARGETS

FirebaseAppCheck/Sources/DeviceCheckProvider/FIRDeviceCheckProviderFactory.m

@@ -14,10 +14,6 @@
  * limitations under the License.
  */
 
-#import "FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRAppCheckAvailability.h"
-
-#if FIR_DEVICE_CHECK_SUPPORTED_TARGETS
-
 #import "FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRDeviceCheckProviderFactory.h"
 
 #import "FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRAppCheck.h"
@@ -34,5 +30,3 @@ + (void)load {
 }
 
 @end
-
-#endif // FIR_DEVICE_CHECK_SUPPORTED_TARGETS

FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRAppCheckAvailability.h

@@ -22,6 +22,8 @@
 #pragma mark - DeviceCheck
 
 // Targets where DeviceCheck framework is available to be used in preprocessor conditions.
+// TODO: Remove this macro during a breaking change; it is no longer used in the codebase but is
+// defined in the public API.
 #define FIR_DEVICE_CHECK_SUPPORTED_TARGETS \
  TARGET_OS_IOS || TARGET_OS_OSX || TARGET_OS_TV || TARGET_OS_WATCH
 
@@ -32,6 +34,8 @@
 #pragma mark - App Attest
 
 // Targets where `DCAppAttestService` is available to be used in preprocessor conditions.
+// TODO: Remove this macro during a breaking change; it is no longer used in the codebase but is
+// defined in the public API.
 #define FIR_APP_ATTEST_SUPPORTED_TARGETS \
  TARGET_OS_IOS || TARGET_OS_OSX || TARGET_OS_TV || TARGET_OS_WATCH
 

FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRDeviceCheckProvider.h

@@ -19,8 +19,6 @@
 #import "FIRAppCheckAvailability.h"
 #import "FIRAppCheckProvider.h"
 
-#if FIR_DEVICE_CHECK_SUPPORTED_TARGETS
-
 @class FIRApp;
 @protocol FIRDeviceCheckAPIServiceProtocol;
 @protocol FIRDeviceCheckTokenGenerator;
@@ -46,5 +44,3 @@ NS_SWIFT_NAME(DeviceCheckProvider)
 @end
 
 NS_ASSUME_NONNULL_END
-
-#endif // FIR_DEVICE_CHECK_SUPPORTED_TARGETS

FirebaseAppCheck/Sources/Public/FirebaseAppCheck/FIRDeviceCheckProviderFactory.h

@@ -14,12 +14,9 @@
  * limitations under the License.
  */
 
-#import "FIRAppCheckAvailability.h"
-
-#if FIR_DEVICE_CHECK_SUPPORTED_TARGETS
-
 #import <Foundation/Foundation.h>
 
+#import "FIRAppCheckAvailability.h"
 #import "FIRAppCheckProviderFactory.h"
 
 NS_ASSUME_NONNULL_BEGIN
@@ -37,5 +34,3 @@ NS_SWIFT_NAME(DeviceCheckProviderFactory)
 @end
 
 NS_ASSUME_NONNULL_END
-
-#endif // FIR_DEVICE_CHECK_SUPPORTED_TARGETS