[webkit-reviews] review requested: [Bug 35503] [Qt] Implement Desktop Notifications API for QtWebKit : [Attachment 51906] 2nd try

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Mar 29 07:14:34 PDT 2010


Laszlo Gombos <laszlo.1.gombos at nokia.com> has asked  for review:
Bug 35503: [Qt] Implement Desktop Notifications API for QtWebKit
https://bugs.webkit.org/show_bug.cgi?id=35503

Attachment 51906: 2nd try
https://bugs.webkit.org/attachment.cgi?id=51906&action=review

------- Additional Comments from Laszlo Gombos <laszlo.1.gombos at nokia.com>
Thanks Eric for the review. 

Good catch on the leak. I changed the patch so that now each instance QWebPage
of owns (creates and destroys) an instance of NotificationPresenterClientQt.


More information about the webkit-reviews mailing list