Ignore failures for notifyLocalViewChanges

[schmidt-sebastian]

This adds code to ignore failures during notifyLocalViewChanges. This should only affect LRU GC and only mean that the documents get GCed earlier.

If we have to add recovery logic we probably need to buffer these changes and apply them via enqueueRetryable before running LRU GC. That's why I am hoping that we don't need to.

Disclaimer: I am not sure if this is actually correct, but I hope it is.

Addresses #2755

[google-oss-bot]

Binary Size Report

Affected SDKs

• @firebase/firestore

  Type	Base (f3f1b17)	Head (9859d12)	Diff
  browser	249 kB	250 kB	+315 B (+0.1%)
  esm2017	194 kB	194 kB	+111 B (+0.1%)
  main	490 kB	490 kB	+409 B (+0.1%)
  module	247 kB	247 kB	+293 B (+0.1%)

• @firebase/firestore/memory

  Type	Base (f3f1b17)	Head (9859d12)	Diff
  browser	190 kB	190 kB	+315 B (+0.2%)
  esm2017	148 kB	148 kB	+111 B (+0.1%)
  main	366 kB	367 kB	+407 B (+0.1%)
  module	188 kB	188 kB	+293 B (+0.2%)

• firebase

  Type	Base (f3f1b17)	Head (9859d12)	Diff
  firebase-firestore.js	288 kB	288 kB	+291 B (+0.1%)
  firebase-firestore.memory.js	230 kB	230 kB	+291 B (+0.1%)
  firebase.js	821 kB	822 kB	+293 B (+0.0%)

Test Logs

• Base (f3f1b173): https://github.com/firebase/firebase-js-sdk/actions/runs/104196948
• Head (9859d125): https://github.com/firebase/firebase-js-sdk/actions/runs/104319186

[wilhuff]

LGTM