RenderWrapper was meant to be a stop-gap to remove the process layer. It served it's purpose.
The idea is that once assetic was in there would be no need for this.and being removed here :( Well assetic is taking it's time, and there is no need for it now!
Though while working on auto-escape on by default in twig @chx and I found that RenderWrapper caused all kinds of PITA issues and exception workarounds.
A few things that used to need RenderWrapper and there are just a few stragglers, namely drupal_get_js() and drupal_get_css() and a closure for meta and links.
Also I hate this class name with a passion, I came up with it and it's haunting me! Hated it also when it was created too. So please, please remove this
poorly named magic wonder box of shame
Add helpers to HTMLFragment and call those helpers from the template.
Remove the class.
Write a patch
- Cleanup coding standards, docs.
- Write change record.
User interface changes
Removal of silly wonder box. (aka, "I wonder why they called it that, I don't know what the **** it does...").
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 72,866 pass(es). View
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 2272279_34.patch. Unable to apply patch. See the log in the details link for more information. View
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 2272279-die-RenderWrapper-19.patch. Unable to apply patch. See the log in the details link for more information. View