----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101140/#review2680 -----------------------------------------------------------
Wouldn't it make more sense to change KProtocolInfo::protocolClass() such that it considers data: to be local access? - Kevin On April 16, 2011, 4:27 p.m., Volker Krause wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101140/ > ----------------------------------------------------------- > > (Updated April 16, 2011, 4:27 p.m.) > > > Review request for kdelibs. > > > Summary > ------- > > Currently KIO::AccessManager blocks retrieval of embedded data: URLs if > external references are disabled. This does not match the behavior in KHTML > and breaks for example the display of sender photos/logos in KMail (which > uses kdewebkit). > > > Diffs > ----- > > kio/kio/accessmanager.cpp bfb4721 > > Diff: http://git.reviewboard.kde.org/r/101140/diff > > > Testing > ------- > > > Thanks, > > Volker > >