diff options
| author |  Michael Lehenbauer <mikelehen@gmail.com> | 2018-03-21 14:59:49 -0700 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2018-03-21 14:59:49 -0700 |
| commit | 5f49b2f3f9866e4db13d09857eb3b548239cc62e (patch) | |
| tree | f1490f36966b49b73c4fe2e9373b86809db383ec /Firestore/Source/Util | |
| parent | 7854c5164b4440201514b5ab0d90554dd94e9455 (diff) | |
Fix for b/74749605: Cancel pending backoff operations when closing streams. (#958)
Diffstat (limited to 'Firestore/Source/Util')
| -rw-r--r-- | Firestore/Source/Util/FSTDispatchQueue.mm | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/Firestore/Source/Util/FSTDispatchQueue.mm b/Firestore/Source/Util/FSTDispatchQueue.mm index 52482df..15d6e7b 100644 --- a/Firestore/Source/Util/FSTDispatchQueue.mm +++ b/Firestore/Source/Util/FSTDispatchQueue.mm @@ -230,11 +230,8 @@ NS_ASSUME_NONNULL_BEGIN block:(void (^)(void))block { // While not necessarily harmful, we currently don't expect to have multiple callbacks with the // same timerID in the queue, so defensively reject them. - // TODO(b/74749605): If a user change happens while offline we can end up with multiple backoff - // callbacks in the dispatch queue. This is non-harmful so I'm just disabling the assert until we - // get that cleaned up. - // FSTAssert(![self containsDelayedCallbackWithTimerID:timerID], - // @"Attempted to schedule multiple callbacks with id %ld", (unsigned long)timerID); + FSTAssert(![self containsDelayedCallbackWithTimerID:timerID], + @"Attempted to schedule multiple callbacks with id %ld", (unsigned long)timerID); FSTDelayedCallback *delayedCallback = [FSTDelayedCallback createAndScheduleWithQueue:self timerID:timerID delay:delay |