Bug 1098168 - BaseIcon

So, I see that icons which have settings listeners mixin the BaseModule, but it still seems weird to me that we clear `this._settings` in BaseIcon since it has no notion of `_settings`.