shell: Move cockpit_format_bytes into cockpit namespace

And make the function easier to use correctly

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
shell: Put cockpit-storage code in own private scope

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
shell: Move the cockpit_format_bytes_long into storage code

This is where it's used from and it's storage specific.

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
shell: Move cockpit.format_bytes_per_sec into cockpit scope

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
shell: Move cockpit.format_delay into cockpit scope

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
shell: Remove unused cockpit_add_thousands_separater function

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
shell: Move cockpit_format_temperature into storage code

It's currently disk specific, and so scope it appropriately.

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
shell: bits rather than bytes to describe network throughput

Refactor how we build up the suffixes.

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
Fixes #855
Closes #877