Code review comment for lp://staging/~mvo/software-center/lp971776

Revision history for this message
Gary Lasker (gary-lasker) wrote :

Looks great and should do the trick nicely! Thanks for adding the unit test for this error case. I also tested by clearing the local icon cache, everything works like a charm.

Thanks!

review: Approve

« Back to merge proposal