From 8bffacb2ba8eb365013223e77a79afebdca53d3f Mon Sep 17 00:00:00 2001
From: Brian Gough <briangough@users.noreply.github.com>
Date: Mon, 9 May 2022 14:07:25 +0100
Subject: [PATCH] Merge pull request #7845 from
 overleaf/bg-snapshot-lost-tracked-changes

[document-updater] consider lost tracked-changes as collapsed ranges

GitOrigin-RevId: cf8e15c610f906f1296fdcdd69128ad1e029863a
---
 services/document-updater/app/js/RangesManager.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/services/document-updater/app/js/RangesManager.js b/services/document-updater/app/js/RangesManager.js
index defba60e24..581a7426c8 100644
--- a/services/document-updater/app/js/RangesManager.js
+++ b/services/document-updater/app/js/RangesManager.js
@@ -80,7 +80,9 @@ module.exports = RangesManager = {
 
     const [emptyRangeCountAfter, totalRangeCountAfter] =
       RangesManager._emptyRangesCount(rangesTracker)
-    const rangesWereCollapsed = emptyRangeCountAfter > emptyRangeCountBefore
+    const rangesWereCollapsed =
+      emptyRangeCountAfter > emptyRangeCountBefore ||
+      totalRangeCountAfter + 1 < totalRangeCountBefore // also include the case where multiple ranges were removed
     // monitor the change in range count, we may want to snapshot before large decreases
     if (totalRangeCountAfter < totalRangeCountBefore) {
       Metrics.histogram(