MessageSender::send_contact_details seems to have some parameters too many

[rubdos]

I'm working on #206, and I'm adding the include_pni_signature bool everywhere. I notice that send_contact_details has the online parameter and the recipient parameter, but the former should always be false, and the latter is always the self-recipient. So we'll want to clean that out.

Or am I missing something?