Replace theme() with drupal_render() in views.module

Retitling

I've reviewed this patch a couple times now, looks great to me. I think careful reviews will be more beneficial than manual testing for these patches and had a quick chat with @thedavidmeister about this on IRC just now. I will be updating the issue summary on the meta issue slightly to make the scope of these conversions a bit smaller.

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/display/DisplayPluginBase.phpundefined
@@ -1848,7 +1849,10 @@ public function buildOptionsForm(&$form, &$form_state) {
-          '#markup' => '<div class="form-item">' . theme('item_list', array('items' => $funcs)) . '</div>',
+          '#theme' => 'item_list',
+          '#prefix' => '<div class="form-item">',
+          '#items' => $funcs,
+          '#suffix' => '</div>',

Okay discussed on IRC, at least this bit could use some manual testing :)

Taked for manual testing #2009672

Behaviour in views ui didn't change. Patch working fine.

Committed/pushed to 8.x, thanks!

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/display/DisplayPluginBase.phpundefined
@@ -2083,8 +2087,11 @@ protected function formatThemes($themes) {
-    return theme('item_list', array('items' => array_reverse($fixed)));
+    $item_list = array(
+      '#theme' => 'item_list',
+      '#items' => array_reverse($fixed),
+    );
+    return drupal_render($item_list);

let's not do unnecessary assignments... just do

  return(drupal_render(array(
    '#theme' => 'item_list',
    '#items' => array_reverse($fixed),
  )));

Therefore...

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/field/PrerenderList.phpundefined
@@ -79,12 +79,13 @@ function render_items($items) {
-        return theme('item_list',
-          array(
-            'items' => $items,
-            'title' => NULL,
-            'type' => $this->options['type']
-          ));
+        $item_list = array(
+          '#theme' => 'item_list',
+          '#items' => $items,
+          '#title' => NULL,
+          '#type' => $this->options['type'],
+        );
+        return drupal_render($item_list);

Same here...

+++ b/core/modules/views/views.theme.incundefined
@@ -1181,14 +1185,16 @@ function theme_views_mini_pager($vars) {
-  return theme('item_list',
-    array(
-      'items' => $items,
-      'title' => NULL,
-      'type' => 'ul',
-      'attributes' => array('class' => array('pager')),
-    )
+  $item_list = array(
+    '#theme' => 'item_list',
+    '#items' => $items,
+    '#title' => NULL,
+    '#type' => 'ul',
+    '#attributes' => array(
+      'class' => array('pager'),
+    ),
   );
+  return drupal_render($item_list);

Same here...

Awesome cross post... back to fixed then...

drupal_render() takes the element by reference anyway, we needed those assignments.
Awesome cross post indeed.