Cleanup of some run view code before larger changes.

Now properly uses local variables in the partials and other cosmetic changes.
[SERVINF-117] Streamline workflow searching in run views.

Now just got once from the database in the controller, not the view.
Get results in controller and only render when there are any.

Results are collected in the controller (although at this point they are still
also being searched for in the view as well) and the results partial is only
now rendered if there are any results.
[SERVINF-115] Reduce database access for results display.

Also remove the code in the views that performed database accesses!
Fixup various formatting issues in the taverna updater worker code.

So they don't get confused with actual updates.
[SERVINF-122] Store list output mimetypes correctly.

These were just being stored with the first port mimetype in the list for all
list items.

This would cause issues if there were different types in the list. While this
would not really matter for text outputs it would be a problem with mixed text
and image lists.
[SERVINF-121] Show list indices for list output items.

Also give list outputs a title to match non-list outputs and show the mimetypes
of each list value.
[SERVINF-118] Don't keep getting username from database.

We already have the username in current_user, so use that and certainly stop
getting it from within the Run view!
Code cleanup in the run index view.

Do it in a separate commit so we don't obscure real updates.
[SERVINF-116] Remove result count from run list.

This needed a database access per run, so on the portal where there are a lot
of runs (100's) this took a long time.
[SERVINF-119/120] Improve browser recommendation message.

Remove the Firefox logo and better explain the browsers that we support.