Remove AbstractPipelineViewAction

[lewisbirks]

There was only one implementation of AbstractPipelineViewAction, as such I think it is better if we inline into its only usage, PipelineConsoleViewAction.

I've grouped some of the icon methods that at first appeared unused but were in fact used by jelly files in core.

I also noticed that replayRun method was unused and as such have removed it.

Testing done

A quick visual check has been done and relying on existing tests.

Submitter checklist

• Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
• Ensure that the pull request title represents the desired changelog entry
• Please describe what you did
• Link to relevant issues in GitHub or Jira
• Link to relevant pull requests, esp. upstream and downstream changes
• Ensure you have provided tests that demonstrate the feature works or the issue is fixed

[lewisbirks]

do these need to be transient? they were in the base but the PipelineStepApi isn't and the other copy of the field, target, wasn't originally

[timja]

as long as the action isn't persisted then no they don't need to be transient, should be fine to remove.