Agree, so you can replace the ```GetIconExtension(icon_name_)``` by ```base::FilePath::FromUTF8Unsafe(icon_name_).Extension()```, and then, GetIconExtension will have no used anymore.