(WIP)(PUP-3276) Remove deprecated methods from Puppet::Util::Windows::*

This commit removes deprecated methods from several Windows utility
modules. Removed methods include:

* `Puppet::Util::Windows::ADSI.sid_for_account`
* `Puppet::Util::Windows::ADSI::Group#add_members`
* `Puppet::Util::Windows::ADSI::Group#remove_members`
* `Puppet::Util::Windows::File.get_file_attributes`
* `Puppet::Util::Windows::Security.get_attributes`
* `Puppet::Util::Windows::Security.add_attributes`
* `Puppet::Util::Windows::Security.remove_attributes`
* `Puppet::Util::Windows::Security.set_attributes`
* `Puppet::Util::Windows::Security.with_process_token`
* `Puppet::Util::Windows::Security.name_to_sid`
* `Puppet::Util::Windows::Security.name_to_sid_object`
* `Puppet::Util::Windows::Security.octet_string_to_sid_object`
* `Puppet::Util::Windows::Security.sid_to_name`
* `Puppet::Util::Windows::Security.sid_ptr_to_string`
* `Puppet::Util::Windows::Security.string_to_sid_ptr`
* `Puppet::Util::Windows::Security.valid_sid?`

This commit is part of the code removal effort for Puppet 4.
@ferventcoder do you think this is redundant since the test for `sid_uri` already checks for this? Added it to try and match how we tested `add_members` for the newer `add_member_sids`