rails/rails — pull request #8261 by steveklabnik (2012-11-18 14:13:21+00:00)
Title: Simplify String#mb_chars and fix documentation.

Opened for feedback.

This came up due to #7798. I figured that if the if was a 1.8 to 1.9 difference, we should not just fix docs, but remove the if. Now one test fails:

```
1) Failure:
test_methods_are_forwarded_to_wrapped_string_for_byte_strings(MultibyteCharsTest) [/Users/steve/src/rails/activesupport/test/multibyte_chars_test.rb:50]:
--- expected
+++ actual
@@ -1 +1 @@
-String
+ActiveSupport::Multibyte::Chars
```

I'm not sure this test is good, though: do we really want to proxy `.class`?