MainContentViewSubscriber should use a service locator

 5 files changed, 18 insertions(+), 73 deletions(-)

This is an event subscriber and I doubt anyone would be overriding it (you would swap out the renderer service instead) and so I don't feel we need to provide BC. We could deprecate the unused compiler pass instead of deleting it outright if that is deemed necessary.

Since this file wasn't marked @internal any backwards compatibility concerns with deleting?

Will try and add some BC.