I think it may be safe to get rid of the first half of the condition now. If newest is null, snapshotFragments is either empty or no viable snapshot was found.

The if block below is also unreachable.

if (snapshotFragments.isEmpty()) {
    return null;
}