(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 these actually look to already be covered in spec/unit/util/windows/adsi_spec.rb, lines 242-261, except testing that an error gets raised on an invalid SID, which I've added below.