Remove references to non existing preprocess functions

The patch attached removes all the references to template_preprocess_status_messages().

Removes the following references:
template_preprocess_config_translation_manage_form_element
template_preprocess_dropbutton_wrapper
template_preprocess_menu_local_tasks
template_preprocess_page_title
template_preprocess_tablesort_indicator
template_preprocess_update_version
template_preprocess_views_ui_container

Patch is attached. I think there's no references left in template files.

This is pretty good I did notice bartik is referencing bartik_preprocess_page() which doesn't exist. Could you do a search through all @see's in twig templates to see the functions exist?

Here's a grep to get a list of all:
grep -ihREo --include="*.twig" '(\w*)\(\)' core | sort | uniq

And to get the file names they are in:
grep -iREo --include="*.twig" '(\w*)\(\)' core | sort

Removing following references:
template_preprocess_page_title()
template_preprocess_views_ui_container()
template_preprocess_update_version()
bartik_preprocess_page()
bartik_preprocess_field()
template_preprocess_views_ui_container()
template_preprocess_update_version()

Found two additions. I think this should be all of them, I compared lists of undefined ones with the ones in the patch and only came up with these. Looks good to me otherwise.

Nice hunting

Back to black

The bartik patch seems to conflict with #2613998: Remove bartik_preprocess_page() reference from page.html.twig

Working on it now.

Rerolled.

Perfect, I diffed the diffs and the re-roll looks great.

Committed/pushed to 8.1.x and cherry-picked to 8.0.x. Thanks!

Cleaning up drupalcampfi tags.