Fix duplication of recoverable list changes after unrecoverable changes

[tgoyne]

After we copy a list in client reset recovery we need to not apply any changes to that list in subsequent transactions as we copy directly to the final state of the list and applying changes would just duplicate those changes.

This is only applicable to flexible sync when there's a subscription change in between the write with unrecoverable changes and the write with recoverable changes.

[coveralls-official]

Pull Request Test Coverage Report for Build github_pull_request_285434

• 77 of 77 (100.0%) changed or added relevant lines in 3 files are covered.
• 68 unchanged lines in 13 files lost coverage.
• Overall coverage increased (+0.008%) to 91.697%

---

Files with Coverage Reduction	New Missed Lines	%
src/realm/util/compression.cpp	1	91.59%
test/test_index_string.cpp	1	94.13%
src/realm/object-store/shared_realm.cpp	2	93.74%
src/realm/sync/network/websocket.cpp	2	74.23%
src/realm/util/file.cpp	2	82.05%
test/test_lang_bind_helper.cpp	2	93.3%
src/realm/sync/network/network.cpp	3	89.86%
src/realm/sync/noinst/client_reset.cpp	4	93.5%
src/realm/sync/noinst/client_reset_operation.cpp	4	90.32%
src/realm/sync/transform.cpp	4	63.08%

Totals
Change from base Build 1854:	0.008%
Covered Lines:	231370
Relevant Lines:	252321

---

💛 - Coveralls

[ironage]

Good catch 👍
Copying lists was implemented before FLX recovery where multiple transactions were possible, and this was missed.