The implementation of `IWpfTextView` is not thread safe and it seems wrong to return it from this function (given the lock I'm presuming this can be called from multiple threads).  It would seem safer to return `ITextBuffer` or `ITextSnapshot` here instead.  Is there a reason that we must use `IWpfTextView` here?