This is a follow up from #32503 where IFileService.updateContent() now accepts a ITextSnapshot to prevent loading the entire buffer into memory. I left a TODO here to use ITextSnapshot directly and not the current fallback snapshotToString