Review Request 114253: Port favicons dataengine to KF5

[Andrea Scarpino]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114253/
-----------------------------------------------------------

(Updated Dec. 5, 2013, 7:27 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Repository: kde-workspace


Description
-------

I replaced KUrl by QUrl and KStandardDirs by QStandardPaths. Also I cleanup includes.

The url given in input could be absolute or relative but not local, then I used QUrl::fromUserInput().


Diffs
-----

  plasma/generic/dataengines/favicons/favicons.cpp 2eae673 
  plasma/generic/dataengines/favicons/favicons.h 79565bf 
  plasma/generic/dataengines/favicons/faviconprovider.cpp bc72a66 
  plasma/generic/dataengines/favicons/faviconprovider.h 1f370f9 
  plasma/generic/dataengines/CMakeLists.txt 3e94325 
  plasma/generic/dataengines/favicons/CMakeLists.txt 4af0f14 

Diff: http://git.reviewboard.kde.org/r/114253/diff/


Testing
-------

If I try to fetch several favicons an empty image is returned, except the first one which contains a valid favicon.


Thanks,

Andrea Scarpino

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131205/3b822af3/attachment.html>