Thanks for pointing this out. Basically you're right.

It was added to keep tests green while making all those changes. Either way, `ActionMailer` is right now in "broken state" which means it has a lot of failures in test suite.

I decided to deal with AP<=>AV part first. Once I'll work out all issues here, we'll have consensus on API and all changes, I'll take care of AM. It should be fairly easy to "fix it" and adjust to new changes.

Long story short.

Yes. I think that ActionMailer should not depend on ActionView.