Convert most of the left over local tasks

Don't we need a parent_id in the SearchLocalTask local tasks?

#2032309: Use local tasks derivatives to provide local tasks for views would get this even closer.

Do we want to clear up MENU_SIBLING_LOCAL_TASK here too?

+++ b/core/modules/search/lib/Drupal/search/Form/SearchSettingsForm.php
@@ -263,6 +263,7 @@ public function submitForm(array &$form, array &$form_state) {
+      \Drupal::cache('cache')->deleteTags(array('local_task' => 1));

This should be TRUE instead of 1. The tag will get casted to a 1 when the tags are flattened, but this should be TRUE. It's just this particular tag storage that does this.

Great, thanks! That looks good now. Will RTBC when green.

note for others regarding #13, we can't do that because we still need to iterate through all cache bins to clear tags.

+++ b/core/modules/entity/entity.local_tasks.yml
@@ -7,3 +7,18 @@ entity.form_mode_edit:
+entity.view_mode_list:
+  title: List
+  route_name: entity.view_mode_list
+  tab_root_id: entity.view_mode_list
+
+entity.view_mode_list:
+  title: List
+  route_name: entity.view_mode_list
+  tab_root_id: entity.view_mode_list

This looks duplicated.

Looks good now :)

Awesome! Glad to see these picked off. From IRC, sounds like we're very close now outside of menu_test.module.

Although this...

+++ b/core/modules/search/lib/Drupal/search/Plugin/Derivative/SearchLocalTask.php
@@ -0,0 +1,42 @@
+/**
+ * Provides local tasks for each search plugin.
+ */
+class SearchLocalTask extends DerivativeBase {
+

Wow, that's quite a mouthful. :\ I guess that's what #2092529: [meta] Improve DX for defining custom routes is attempting to clean up?

In any event...

Committed and pushed to 8.x. Thanks!