Removed unnecessary call to 'convert_key' in 'HashWithIndifferentAccess#to_hash'

Removed unnecessary call to `convert_key` in `HashWithIndifferentAccess#to_hash` introduced in df24b8790f22384a068fece7042f04ffd2fcb33e

All the keys are already Strings by virtue of being a `HashWithIndifferentAccess`.

I ran a simple benchmark and observed a 4% performance increase when calling the `to_hash` method.

    Benchmark.ips do |x|
      x.time = 30
      x.warmup = 5

      hash = {
        :a  => 'a',
        'b' => { :b1 => 'b1', 'b2' => { 'b3' => 'b3' } },
        :c  => 'c'
      }.with_indifferent_access

      x.report("to_hash") do
        hash.to_hash
      end
    end

You're correct, that would work. That would save creating another `Hash` object.

Here is another solution

```ruby
    def to_hash
      each_with_object(Hash.new(default)) do |(key, value), hash|
        hash[key] = convert_value(value, for: :to_hash)
      end
    end
```