Reverts "Timing: Fixes timer when app get into background (#24649)" (#27073)

Summary:
This PR reverts commit 3382984 that is causing #26696 #26995.
> app would be closed immediately after going to background on iOS 13.1/13.2 and was investigated by minhtc #26696 (comment). The commit that is being reverted is apparently causing the app to be closed immediately. This has to be reverted in master separately as the file differs there.

Similar PR for 0.61. branch #27065

## Changelog

[iOS] [Fixed] - Fix apps crashing on iOS 13.x when running timer in the background
Pull Request resolved: #27073

Test Plan: Try [this](3382984#commitcomment-35745287) snippet on iOS 13.1/13.2, the app should not crash anymore

Differential Revision: D18323679

Pulled By: cpojer

fbshipit-source-id: 3af7036a0e1d3811924e581c649b16e5a4667e83

Author: radko93

React/CoreModules/RCTTiming.mm

@@ -12,9 +12,9 @@
 #import <React/RCTAssert.h>
 #import <React/RCTBridge+Private.h>
 #import <React/RCTBridge.h>
+#import <React/RCTConvert.h>
 #import <React/RCTLog.h>
 #import <React/RCTUtils.h>
-#import <React/RCTConvert.h>
 
 #import "CoreModulesPlugins.h"
 
@@ -75,8 +75,7 @@ @interface _RCTTimingProxy : NSObject
 @end
 
 // NSTimer retains its target, insert this class to break potential retain cycles
-@implementation _RCTTimingProxy
-{
+@implementation _RCTTimingProxy {
   __weak id _target;
 }
 
@@ -96,16 +95,14 @@ - (void)timerDidFire
 
 @end
 
-@interface RCTTiming() <NativeTimingSpec>
+@interface RCTTiming () <NativeTimingSpec>
 @end
 
-@implementation RCTTiming
-{
+@implementation RCTTiming {
   NSMutableDictionary<NSNumber *, _RCTTimer *> *_timers;
   NSTimer *_sleepTimer;
   BOOL _sendIdleEvents;
   BOOL _inBackground;
-  UIBackgroundTaskIdentifier _backgroundTaskIdentifier;
   id<RCTTimingDelegate> _timingDelegate;
 }
 
@@ -115,7 +112,7 @@ @implementation RCTTiming
 
 RCT_EXPORT_MODULE()
 
-- (instancetype)initWithDelegate:(id<RCTTimingDelegate>) delegate
+- (instancetype)initWithDelegate:(id<RCTTimingDelegate>)delegate
 {
   if (self = [super init]) {
     [self setup];
@@ -136,19 +133,19 @@ - (void)setup
   _paused = YES;
   _timers = [NSMutableDictionary new];
   _inBackground = NO;
-  _backgroundTaskIdentifier = UIBackgroundTaskInvalid;
 
-  for (NSString *name in @[UIApplicationWillResignActiveNotification,
-                           UIApplicationDidEnterBackgroundNotification,
-                           UIApplicationWillTerminateNotification]) {
+  for (NSString *name in @[
+         UIApplicationWillResignActiveNotification,
+         UIApplicationDidEnterBackgroundNotification,
+         UIApplicationWillTerminateNotification
+       ]) {
     [[NSNotificationCenter defaultCenter] addObserver:self
                                              selector:@selector(appDidMoveToBackground)
                                                  name:name
                                                object:nil];
   }
 
-  for (NSString *name in @[UIApplicationDidBecomeActiveNotification,
-                           UIApplicationWillEnterForegroundNotification]) {
+  for (NSString *name in @[ UIApplicationDidBecomeActiveNotification, UIApplicationWillEnterForegroundNotification ]) {
     [[NSNotificationCenter defaultCenter] addObserver:self
                                              selector:@selector(appDidMoveToForeground)
                                                  name:name
@@ -158,34 +155,9 @@ - (void)setup
 
 - (void)dealloc
 {
-  [self markEndOfBackgroundTaskIfNeeded];
   [_sleepTimer invalidate];
 }
 
-- (void)markStartOfBackgroundTaskIfNeeded
-{
-  if (_backgroundTaskIdentifier == UIBackgroundTaskInvalid) {
-    __weak RCTTiming *weakSelf = self;
-    // Marks the beginning of a new long-running background task. We can run the timer in the background.
-    _backgroundTaskIdentifier = [[UIApplication sharedApplication] beginBackgroundTaskWithName:@"rct.timing.gb.task" expirationHandler:^{
-      RCTTiming *strongSelf = weakSelf;
-      if (!strongSelf) {
-        return;
-      }
-      // Mark the end of background task
-      [strongSelf markEndOfBackgroundTaskIfNeeded];
-    }];
-  }
-}
-
-- (void)markEndOfBackgroundTaskIfNeeded
-{
-  if (_backgroundTaskIdentifier != UIBackgroundTaskInvalid) {
-    [[UIApplication sharedApplication] endBackgroundTask:_backgroundTaskIdentifier];
-    _backgroundTaskIdentifier = UIBackgroundTaskInvalid;
-  }
-}
-
 - (dispatch_queue_t)methodQueue
 {
   return RCTJSThread;
@@ -211,7 +183,6 @@ - (void)appDidMoveToBackground
 
 - (void)appDidMoveToForeground
 {
-  [self markEndOfBackgroundTaskIfNeeded];
   _inBackground = NO;
   [self startTimers];
 }
@@ -246,7 +217,7 @@ - (void)startTimers
 
 - (BOOL)hasPendingTimers
 {
-  @synchronized (_timers) {
+  @synchronized(_timers) {
     return _sendIdleEvents || _timers.count > 0;
   }
 }
@@ -256,7 +227,7 @@ - (void)didUpdateFrame:(RCTFrameUpdate *)update
   NSDate *nextScheduledTarget = [NSDate distantFuture];
   NSMutableArray<_RCTTimer *> *timersToCall = [NSMutableArray new];
   NSDate *now = [NSDate date]; // compare all the timers to the same base time
-  @synchronized (_timers) {
+  @synchronized(_timers) {
     for (_RCTTimer *timer in _timers.allValues) {
       if ([timer shouldFire:now]) {
         [timersToCall addObject:timer];
@@ -272,7 +243,7 @@ - (void)didUpdateFrame:(RCTFrameUpdate *)update
       return [a.target compare:b.target];
     }] valueForKey:@"callbackID"];
     if (_bridge) {
-      [_bridge enqueueJSCall:@"JSTimers" method:@"callTimers" args:@[sortedTimers] completion:NULL];
+      [_bridge enqueueJSCall:@"JSTimers" method:@"callTimers" args:@[ sortedTimers ] completion:NULL];
     } else {
       [_timingDelegate callTimers:sortedTimers];
     }
@@ -283,7 +254,7 @@ - (void)didUpdateFrame:(RCTFrameUpdate *)update
       [timer reschedule];
       nextScheduledTarget = [nextScheduledTarget earlierDate:timer.target];
     } else {
-      @synchronized (_timers) {
+      @synchronized(_timers) {
         [_timers removeObjectForKey:timer.callbackID];
       }
     }
@@ -294,25 +265,23 @@ - (void)didUpdateFrame:(RCTFrameUpdate *)update
     NSTimeInterval frameElapsed = currentTimestamp - update.timestamp;
     if (kFrameDuration - frameElapsed >= kIdleCallbackFrameDeadline) {
       NSNumber *absoluteFrameStartMS = @((currentTimestamp - frameElapsed) * 1000);
-      if (_bridge){
-        [_bridge enqueueJSCall:@"JSTimers" method:@"callIdleCallbacks" args:@[absoluteFrameStartMS] completion:NULL];
+      if (_bridge) {
+        [_bridge enqueueJSCall:@"JSTimers" method:@"callIdleCallbacks" args:@[ absoluteFrameStartMS ] completion:NULL];
       } else {
         [_timingDelegate callIdleCallbacks:absoluteFrameStartMS];
       }
-
     }
   }
 
   // Switch to a paused state only if we didn't call any timer this frame, so if
   // in response to this timer another timer is scheduled, we don't pause and unpause
   // the displaylink frivolously.
   NSUInteger timerCount;
-  @synchronized (_timers) {
+  @synchronized(_timers) {
     timerCount = _timers.count;
   }
   if (_inBackground) {
     if (timerCount) {
-      [self markStartOfBackgroundTaskIfNeeded];
       [self scheduleSleepTimer:nextScheduledTarget];
     }
   } else if (!_sendIdleEvents && timersToCall.count == 0) {
@@ -331,13 +300,13 @@ - (void)didUpdateFrame:(RCTFrameUpdate *)update
 
 - (void)scheduleSleepTimer:(NSDate *)sleepTarget
 {
-  @synchronized (self) {
+  @synchronized(self) {
     if (!_sleepTimer || !_sleepTimer.valid) {
       _sleepTimer = [[NSTimer alloc] initWithFireDate:sleepTarget
-                                            interval:0
-                                              target:[_RCTTimingProxy proxyWithTarget:self]
-                                            selector:@selector(timerDidFire)
-                                            userInfo:nil
+                                             interval:0
+                                               target:[_RCTTimingProxy proxyWithTarget:self]
+                                             selector:@selector(timerDidFire)
+                                             userInfo:nil
                                               repeats:NO];
       [[NSRunLoop currentRunLoop] addTimer:_sleepTimer forMode:NSDefaultRunLoopMode];
     } else {
@@ -367,13 +336,14 @@ - (void)timerDidFire
  * calculating the timer's target time. We calculate this by passing in
  * Date.now() from JS and then subtracting that from the current time here.
  */
-RCT_EXPORT_METHOD(createTimer:(double)callbackID
-                  duration:(NSTimeInterval)jsDuration
-                  jsSchedulingTime:(double)jsSchedulingTime
-                  repeats:(BOOL)repeats)
+RCT_EXPORT_METHOD(createTimer
+                  : (double)callbackID duration
+                  : (NSTimeInterval)jsDuration jsSchedulingTime
+                  : (double)jsSchedulingTime repeats
+                  : (BOOL)repeats)
 {
   NSNumber *callbackIdObjc = [NSNumber numberWithDouble:callbackID];
-  NSDate *schedulingTime = [RCTConvert NSDate:[NSNumber numberWithDouble: jsSchedulingTime]];
+  NSDate *schedulingTime = [RCTConvert NSDate:[NSNumber numberWithDouble:jsSchedulingTime]];
   if (jsDuration == 0 && repeats == NO) {
     // For super fast, one-off timers, just enqueue them immediately rather than waiting a frame.
     if (_bridge) {
@@ -384,10 +354,7 @@ - (void)timerDidFire
     return;
   }
 
-  [self createTimerForNextFrame:callbackIdObjc
-                       duration:jsDuration
-               jsSchedulingTime:schedulingTime
-                        repeats:repeats];
+  [self createTimerForNextFrame:callbackIdObjc duration:jsDuration jsSchedulingTime:schedulingTime repeats:repeats];
 }
 
 /**
@@ -410,12 +377,11 @@ - (void)createTimerForNextFrame:(nonnull NSNumber *)callbackID
                                                   interval:jsDuration
                                                 targetTime:targetTime
                                                    repeats:repeats];
-  @synchronized (_timers) {
+  @synchronized(_timers) {
     _timers[callbackID] = timer;
   }
 
   if (_inBackground) {
-    [self markStartOfBackgroundTaskIfNeeded];
     [self scheduleSleepTimer:timer.target];
   } else if (_paused) {
     if ([timer.target timeIntervalSinceNow] > kMinimumSleepInterval) {
@@ -426,17 +392,17 @@ - (void)createTimerForNextFrame:(nonnull NSNumber *)callbackID
   }
 }
 
-RCT_EXPORT_METHOD(deleteTimer:(double)timerID)
+RCT_EXPORT_METHOD(deleteTimer : (double)timerID)
 {
-  @synchronized (_timers) {
+  @synchronized(_timers) {
     [_timers removeObjectForKey:[NSNumber numberWithDouble:timerID]];
   }
   if (![self hasPendingTimers]) {
     [self stopTimers];
   }
 }
 
-RCT_EXPORT_METHOD(setSendIdleEvents:(BOOL)sendIdleEvents)
+RCT_EXPORT_METHOD(setSendIdleEvents : (BOOL)sendIdleEvents)
 {
   _sendIdleEvents = sendIdleEvents;
   if (sendIdleEvents) {
@@ -446,13 +412,15 @@ - (void)createTimerForNextFrame:(nonnull NSNumber *)callbackID
   }
 }
 
-- (std::shared_ptr<facebook::react::TurboModule>)getTurboModuleWithJsInvoker:(std::shared_ptr<facebook::react::CallInvoker>)jsInvoker
+- (std::shared_ptr<facebook::react::TurboModule>)getTurboModuleWithJsInvoker:
+    (std::shared_ptr<facebook::react::CallInvoker>)jsInvoker
 {
   return std::make_shared<facebook::react::NativeTimingSpecJSI>(self, jsInvoker);
 }
 
 @end
 
-Class RCTTimingCls(void) {
+Class RCTTimingCls(void)
+{
   return RCTTiming.class;
 }