Code review comment for lp:~azzar1/unity/fix-1036231

Revision history for this message
Andrea Azzarone (azzar1) wrote :

> LGTM. Is this code covered by tests? Should it be?

IconRenderer is test free :/
Btw this is a visual change and we don't need test for visual changes. I'll ask Tim btw.

« Back to merge proposal