Merge pull request #1750 from dmathieu/non_string_methods

Calling unsafe methods which don't return a string shouldn't fail

activesupport/lib/active_support/core_ext/string/output_safety.rb

@@ -137,8 +137,8 @@ module ActiveSupport #:nodoc:
 
     UNSAFE_STRING_METHODS.each do |unsafe_method|
       class_eval <<-EOT, __FILE__, __LINE__
-        def #{unsafe_method}(*args)
-          super.to_str
+        def #{unsafe_method}(*args, &block)
+          to_str.#{unsafe_method}(*args, &block)
         end
 
         def #{unsafe_method}!(*args)

activesupport/test/safe_buffer_test.rb

@@ -104,4 +104,8 @@ class SafeBufferTest < ActiveSupport::TestCase
       @buffer.safe_concat "BUSTED"
     end
   end
+  
+  test "should not fail if the returned object is not a string" do
+    assert_kind_of Enumerator, @buffer.gsub(/.*/)
+  end
 end