BootStrap init closure servletContext cleanup

[gsartori]

See: apache/grails-forge#507 (review)

[jdaugherty]

I am looking at the CI issue

[matrei]

@gsartori I know I'm nagging about this, but why are we injecting an object that is not used?
From our previous discussions around the BootStrap class I have understood that your argument is that it has an educational value on the possibility to do dependency injection in the BootStrap class.
I can buy that argument for the starter app templates, but this is not a starter app template.

[gsartori]

The pattern I've applied while updating the relevant projects was to consider empty BootStrap files as "files from the app template" as they were plain files from the (old) template. I agree with you though, in this context there is an implementation, we can remove the injected proeprty (I went through all the projects in a "lawnmower" mode :)

[jdaugherty]

@gsartori Since we have it as a test in the other cases, can we remove it in this case per @matrei comment? I've merged the other PRs.

[gsartori]

@jdaugherty done