[webkit-reviews] review requested: [Bug 39146] [Qt] Pass all web notification layout tests : [Attachment 56487] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 19 07:31:41 PDT 2010


Yael <yael.aharon at nokia.com> has asked	for review:
Bug 39146: [Qt] Pass all web notification layout tests
https://bugs.webkit.org/show_bug.cgi?id=39146

Attachment 56487: Patch
https://bugs.webkit.org/attachment.cgi?id=56487&action=review

------- Additional Comments from Yael <yael.aharon at nokia.com>
This patch wraps the QSystemTrayIcon, so that we can pass layout tests even if
QT_NO_SYSTEMTRAYICON is not defined.
Qt S60 porting team told me that they could provide a QSystemTrayIcon
impleemntation for Symbian 10.1, so at this time, I don't think we need to hook
up notifications with the platform plugin.

Loading the notifications' icons and markup is still under work.


More information about the webkit-reviews mailing list