QUrl may have an implicit constructor from string, but this should probably be made explicit just to point out that the underlying data is just a string.

return QUrl(d->data().value(FACEBOOK_ONTOLOGY_ALBUM_LINK).toString());