Eliminate FmIcon data, use GObject mechanics on GIcon instead.

The typedef for FmIcon still stays to encourage cache usage but it is
dummy now and GIcon is used instead. That speeds up and lowers memory usage.