Dispose implementations should not be treated as action methods

[rynowak]

Currently if you implement IDisposable on a controller class then your Dispose method will be treated as an action unless you attribute it with [NonAction].

The old controller base classes in both MVC and WebAPI were already IDisposable, but the new one is not, so users will end up defining a dispose method if they need it.

We should special case Dispose to treat it as not-an-action by default.

[pranavkm]

Would Object methods - ToString, GetHashCode also need to be treated likewise?

[rynowak]

We already handle object methods specially

[danroth27]

We should also add Dispose to Controller.

[kanchanm]

We already have Dispose in the Controller so that is a non-issue. We still need to address the POCO controller case though. Working on the fix that will work for the POCO case too.

[kanchanm]

Commit 4598505