Shouldn't have committed this! Revert "avoid calling to_sql twice"

This reverts commit 1c030a3c3c.

activerecord/lib/active_record/relation/query_methods.rb

@@ -135,14 +135,11 @@ module ActiveRecord
         next if where.blank?
 
         case where
-        when Arel::Predicates::In
+        when Arel::SqlLiteral
-          # FIXME: this needs to go away
+          arel = arel.where(where)
-          # when an IN is part of a larger query, the SQL seems to be different
-          arel = arel.where(Arel::SqlLiteral.new("(#{where.to_sql})"))
-        when String
-          arel = arel.where(Arel::SqlLiteral.new("(#{where})"))
         else
-          arel = arel.where where
+          sql = where.is_a?(String) ? where : where.to_sql
+          arel = arel.where(Arel::SqlLiteral.new("(#{sql})"))
         end
       end