Ocean: Accumulate the visibility over all poses in Database::objectPointIds()

Summary: In the `Indices32 poseIds` overload the `tVisibleInAllPoses` test overwrote the flag in every iteration, so only the visibility in `poseIds.back()` decided the outcome. The sibling overload taking a `lowerPoseId`/`upperPoseId` range carries the flag in the loop condition, this one now does the same.

Reviewed By: enpe

Differential Revision: D115133726

fbshipit-source-id: 021cf04ae0100e1ed8dcd44f2e26a87a07b997e6

Author: janherlingmeta

impl/ocean/tracking/Database.h

@@ -4325,7 +4325,7 @@ Indices32 Database::objectPointIds(const Indices32& poseIds, const Vector3& refe
 				if (iO->second.priority() >= minimalPriority && ((tMatchPosition && iO->second.point() == referencePosition) || (!tMatchPosition && iO->second.point() != referencePosition)))
 				{
 					bool visibleInAllPoses = true;
-					for (size_t n = 1; n < poseIds.size(); ++n)
+					for (size_t n = 1; visibleInAllPoses && n < poseIds.size(); ++n)
 						visibleInAllPoses = databasePoseObjectPointMap.find(index64(poseIds[n], objectPointId)) != databasePoseObjectPointMap.end();
 
 					if (visibleInAllPoses)