Fix removal of view references in FSTLocalStore (#3737)

Author: wilhuff

Firestore/Example/Tests/Local/FSTLocalStoreTests.mm

@@ -806,10 +806,10 @@ - (void)testPinsDocumentsInTheLocalView {
   FSTAssertContains(Doc("foo/baz", 2, Map("foo", "baz")));
 
   [self notifyLocalViewChanges:TestViewChanges(targetID, @[], @[ @"foo/bar", @"foo/baz" ])];
-  [self.localStore releaseQuery:query];
-
   FSTAssertNotContains("foo/bar");
   FSTAssertNotContains("foo/baz");
+
+  [self.localStore releaseQuery:query];
 }
 
 - (void)testThrowsAwayDocumentsWithUnknownTargetIDsImmediately {

Firestore/Source/Local/FSTLocalStore.mm

@@ -414,7 +414,7 @@ - (void)notifyLocalViewChanges:(const std::vector<LocalViewChanges> &)viewChange
         [self->_persistence.referenceDelegate removeReference:key];
       }
       _localViewReferences.AddReferences(viewChange.added_keys(), viewChange.target_id());
-      _localViewReferences.AddReferences(viewChange.removed_keys(), viewChange.target_id());
+      _localViewReferences.RemoveReferences(viewChange.removed_keys(), viewChange.target_id());
     }
   });
 }