Use line-column positions instead of character offset for completion requests

This will change completion requests so they use line-column positions instead of character offsets.  I also updated the unit tests (and fixed the Should_complete_diagnostics_only unit test in the process.)

My motivation for this change was that I was experiencing different completion results depending on the EOL char (Unix vs Windows) of the file