As suggested in https://bugs.webkit.org/show_bug.cgi?id=40004, icon loading should be done in cross platform code. This is the cross platform part, and https://bugs.webkit.org/show_bug.cgi?id=40004 would take care of the Qt specific code. A patch is coming soon.
Created attachment 58317 [details] Patch. Make notification objects download the icon needed for displaying the notification before calling the NotificationPresenter to display the notification. An error during the download would cause the notification to be displayed without an icon. If a notification is in the process of download, and a new notification is created with the same ReplaceId, the download is not cancelled immediately and the notification is removed only after the download is complete.
Comment on attachment 58317 [details] Patch. Looks good, r=me
Comment on attachment 58317 [details] Patch. Clearing flags on attachment: 58317 Committed r60960: <http://trac.webkit.org/changeset/60960>
All reviewed patches have been landed. Closing bug.
Revision r60960 cherry-picked into qtwebkit-2.1 with commit 5765325aee8a79e7dedcfa4aa2a984bfb162c63d