Refs #10380 Improve options parsing in Refl UI

We now support quoting values that contain commas, escaping quotes, and
values that contain '='.
refs #10365. Add regression test.

No fix yet. But helps with TDD.
Refs #10380 Generalise parseOptionsString → parseKeyValueString

As a static interface it can now be unit tested appropriately.
refs #10365. Use numeric axis not y values!
Refs #10376 Assign icons to actions
Refs #10376 Move New, Save, and Save As to menu bar
Refs #10376 Change some PushButtons to ToolButtons

* Add Row
* Delete Row
* Group Rows
refs #10365. Missing init call.

We don't need a managed algorithm. Not sure how this ever worked! Later on, I'll create a MantidPlot test for the Rebin2D lineviewer functionality to stop this from falling over again.
Refs #10376 Provide a menu for opening tables

This commit removes the workspace selector in favour of providing a
proper menu for opening tables.
Refs #10376 Update ReflMainViewPresenter unit test

* Add expectations for setTableList
* Remove unnecessary check points. setTableList was being called many
  times after a checkpoint (which had cleared expectations), causing
  many uninteresting calls to setTableList to issue warnings.
Refs #10376 Reduce initial Refl UI size
Merge branch 'master' into feature/10380_refl_ui_options_col_quote_commas
Refs #10380 Unit test parseKeyValueString
Refs #10380 Correct documentation
refs #10365. Fix warnings.
Merge remote-tracking branch 'origin/feature/10365_slice_viewer_crash'
Merge remote-tracking branch 'origin/feature/10376_refl_ui_menu_bar'
Merge remote-tracking branch 'origin/feature/10380_refl_ui_options_col_quote_commas'
Merge branch 'master' into feature/10372_prompt_before_discard_table

Refs #10372

Conflicts:
	Code/Mantid/MantidQt/CustomInterfaces/src/ReflMainViewPresenter.cpp