RESOLVED FIXED Bug 39995
[Qt] Web Notification support 
https://bugs.webkit.org/show_bug.cgi?id=39995
Summary [Qt] Web Notification support
Yael
Reported 2010-06-01 07:29:14 PDT
Web Notification support in QtWebkit is still lacking. This is a meta bug for all the changes I am planning to do. These changes include: 1. Fix the association between Notification object and JSNotification objects. JSNotification objects are not ref'ed properly and are garbage collected shortly after they were created. 2. Make NotificatioPresenter a singleton. A notification object can outlive the page that created it, and the current design does not allow for that. 3. Add support for loading icons for the notifications 4. Add support for the platform plugin to override the default behavior. Things that I did not implement yet: 5. Add support for web workers based notifications. 6. Add support for downloading the markup for HTML based notifications.
Attachments
Add attachment proposed patch, testcase, etc.
Suresh Voruganti
Comment 1 2010-08-25 08:44:29 PDT
Are we expecting any more notification implementation or have finished all implementation tasks? Can we close this bug? this is blocking Bug 39121 Master bug for QtWebkit 2.1
Yael
Comment 2 2010-08-25 14:44:52 PDT
A few things might still be needed for notifications: 1. The spec is still early draft, and changes to it may impact the implementation in QtWebKit. 2. I was asked to hook up Notifications with the Universal Indicator. This will require changes to the PlatformPlugin interface. 3. NotificationPresenter in Chromium is quite different than in QtWebKit, and someone asked me to help consolidate the implementations.
Yael
Comment 3 2010-09-13 05:33:00 PDT
#2 was committed in bug 44836. #1 and #3 are long term goals, so I think after bug 44836 is cherry-picked, we should close this bug.
Ademar Reis
Comment 4 2010-09-13 06:21:39 PDT
(In reply to comment #3) > #2 was committed in bug 44836. #1 and #3 are long term goals, so I think after bug 44836 is cherry-picked, we should close this bug. If all tasks have been done, please go ahead and close it. Patches are cherry-picked when they (and all dependencies) are closed+fixed and block Bug 39121.
Yael
Comment 5 2010-09-13 06:52:50 PDT
All tasks have been done, closing the bug.
Ademar Reis
Comment 6 2010-09-13 07:45:01 PDT
All changes have been integrated into qtwebkit-2.1, removing Bug 39121 from the blockers list. Thanks.
Note You need to log in before you can comment on or make changes to this bug.