Remove ViewEngine property of ViewComponentResult (it doesn't do anything)

[cjqian]

No description provided.

[Eilon]

@cjqian can you provide more info about this? Why it isn't needed anymore? Does the property not work anymore?

[cjqian]

@rynowak

[rynowak]

This wasn't really ever used for anything I don't think.

[Eilon]

Well we can't remove it... Did it just always no-op or something?

[dougbu]

Did it just always no-op or something?

There's exactly (0 + zero + nada) references to this property in the MVC repo. No reads; no writes.

[Eilon]

Hmm, well, we can't remove it due to compatibility, just in case someone is calling it. But in the meantime we can certainly document it as serving no purpose. E.g. "This property is never set and should not be used. It will be removed in a future release." We can mark as Obsolete (warning, not error) with the same message as well.

[Eilon]

I logged #5066 to mark the property as obsolete. Then we'll do the removal (this bug) in 2.0.0.