When shifting & deleting expiredSegment from recentSegments, sort the array first to ensure we are removing the oldest segment

[brandonbothell]

At this rate, I think it might be best to keep track of the last successfully completed motion detection (by segment name) so that we only delete that segment and ones older than it in the segment watcher when motion detection is enabled but motion wasn't detected. I want to ensure we're catching every recent segment before motion started, and not accidentally deleting them due to lagging motion detection.