<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: ngraham, anthonyfieroni.<br />Herald added projects: Dolphin, Frameworks.<br />Herald added subscribers: kfm-devel, kde-frameworks-devel.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15819">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Ico files mostly contain PNGs these days which retain their image header when embedded into an ICO file, so the bits per pixel in the ICO header can be missing or incorrect. Qt still manages to extract the underlying PNG just file but the <tt style="background: #ebebeb; font-size: 13px;">canRead</tt> gives up prematurely.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I now get thumbnails for "sierra chart" exe file</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R320 KIO Extras</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15819">https://phabricator.kde.org/D15819</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>thumbnail/icoutils_common.cpp</div></div></div><br /><div><strong>To: </strong>broulik, ngraham, anthonyfieroni<br /><strong>Cc: </strong>kde-frameworks-devel, kfm-devel, feverfew, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp<br /></div>