Remove uses of deprecated Element functions

I've marked that as a duplicate of this, and I'm stealing its Core Review Bonus tag.

I've not reviewed the patch itself, but those function_replace commands look correct

Needs reroll. I tried, using the commands above, but got

$ function_replace.php element_property 'Drupal\Core\Render\Element' RenderElement property
PHP Fatal error:  Function name must be a string in /Users/ian/d8codetools/src/FileUtil.php on line 19

A few comments needed to be wrapped to 80 chars, but otherwise that's fine. No references to these functions remain, even in comments.

This fixes the wrapping, hopefully it's still OK for me to mark as RTBC.

Setting to needs work, as this is not suitable for commit during Szeged and it's a pretty safe bet that it will conflict with something before the end of the week.

I have spoken to webchick and she suggested that she would be available to review/commit this next Monday, 31st March. There's little point in rerolling the patch until the weekend, but if there's a patch here on Monday that applies then I'll review it.

That's a good reroll. The main changes were done correctly and due to

https://github.com/drupal/drupal/commit/dc15bf2eb231238b601d692da5f59d1b...
https://github.com/drupal/drupal/commit/bead76ad5b8c9e23caf2e5efea2bee01...
https://github.com/drupal/drupal/commit/13242b5af695a0f2aa386ded894a5bc0...

The other changes are just context, except:

+++ b/core/modules/system/lib/Drupal/system/Tests/Common/RenderTest.php
@@ -305,10 +306,10 @@ function testDrupalRenderSorting() {
+    // Pass $elements through \Drupal\Core\Render\Element::children() and
+    // ensure it remains sorted in the correct order. drupal_render() will
+    // return an empty string if used on the same array in the same request.
+    $children = Element::children($elements);

It looks like the script is wrapping comments at 79 characters rather than 80, but the docblock is still easily readable so I'm not going to set needs work for that.