Not quite agree. I think it is redundant, we could get the new icon name by icon_path_.baseName().