Use PathBuf instead of String where appropiate

[Xanewok]

Originally created by @jonasbb (rust-dev-tools/rls-data#7):

CratePreludeData.crate_root and ExternalCrateData.file_name both store file paths, so using PathBuf as the type would semantically make more sense.

[Xanewok]

Originally replied by @nrc (rust-dev-tools/rls-data#7 (comment)):

iirc there was some issues with serialising these types

[Xanewok]

Originally replied by @Xanewok (rust-dev-tools/rls-data#7 (comment)):

These are serialized as an array of bytes in JSON, and is used now for SpanData::file_name.
Is the representation something that would require 'fixing' on the rustc-serialize side?
I assume it's encoded as bytes to handle platform differences with path/string representations but it seems to be working now.
Should we, then, avoid using PathBuf with rustc-serialize altogether or change some Strings to PathBufs and possibly change the JSON representation?

[Xanewok]

Originally replied by @kamalmarhubi (rust-dev-tools/rls-data#7 (comment)):

Assuming that #15 is accepted, moving to serde with custom serialize_with / deserialize_with becomes an option.