Code review comment for lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/unity-theme-icon-provider

> Actually the above is incorrect.
>
> I believe the "name" property should only be left as convenience when used
> directly:
>
> Icon {
> name: "some-known-icon-name"
> }
>
> Whenever the icon url is provided by a backend - it should be a "fully-
> qualified" url, i.e. full image://theme/name path, not just "name", and Icon
> should have a "source" property to deal with those. No one should need to
> guess whether it's a themed icon or a full path. IMO Icon should be renamed to
> "ColourizedIcon" or similar, to denote its real purpose, but Image { } should
> be enough to use, letting the image providers do their thing.

OK, so conclusions from IRC discussions were that theme image provider should only load themed icons. If the developer wants to load absolute or relative icons he should use "file://"