Avoid downcasing facts in service provider

If the os.name or os.family fact is missing this will result in
nil.downcase. In practice that's unlikely, but in some test scenarios it
can happen. By using the properly cased values it removes the need to
downcase, which is also slightly faster.

Author: ekohl

lib/puppet/provider/service/init.rb

@@ -22,9 +22,9 @@ def self.defpath
   # Debian and Ubuntu should use the Debian provider.
   # RedHat systems should use the RedHat provider.
   confine :true => begin
-      os = Puppet.runtime[:facter].value('os.name').downcase
-      family = Puppet.runtime[:facter].value('os.family').downcase
-      !(os == 'debian' || os == 'ubuntu' || family == 'redhat')
+      os = Puppet.runtime[:facter].value('os.name')
+      family = Puppet.runtime[:facter].value('os.family')
+      !(os == 'Debian' || os == 'Ubuntu' || family == 'RedHat')
   end
 
   # We can't confine this here, because the init path can be overridden.