Wouldn't this create the icon during updates instead, eating more memory? I think the icon should be created as a constant instead, so it will be created only on the first instance of the class.