Putting it in "-100" is fine though the magic number's annoying (we should have a PL/[pg]SQL function that returns the global file storage folder).
Oh, wait, we do, look at this!
"create or replace view content_item_globals as
select -100 as c_root_folder_id;"
Just select c_root_folder_id from content_item_globals ... good.
Since we should be moving more and more towards using the template-assigned-to-cr-type mechanism, wouldn't it be better to add a template to the image type now (since someday we'll want this) rather than kludge your code to store the icon as a file-storage-object in a folder that's not managed by file-storage? I don't think its difficult ...
