Clean up PidginCredentialManager

* Remove the unnecessary _provider from a bunch of symbols
* Use g_task_return_new_error
* Some other minor cleanups

Testing Done:
Compiled and ran the unit tests.

Reviewed at https://reviews.imfreedom.org/r/804/