Adding a reference to the filePicker control is a reasonable thing to do in the constructor (though maybe it should be abstracted a bit for better cohesion?). Also, no need to keep the selector AFAIK.
only set the file_browser_callback to reasonplugins when we're in a reasonplugin.