LiveScmController.Delete should take the lock during the deletion

Discussed with David Fowler.  We saw no reason not to take the lock during delete operation.   Upon fixing this, I also removed the code that deletes /diagnostic directory.  This dir holds application diagnostic settings.   We should not delete that with live/scm delete.