[Webkit-unassigned] [Bug 40004] [Qt] Support for loading notification icons

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=40004





--- Comment #6 from Yael <yael.aharon at nokia.com>  2010-06-03 16:30:22 PST ---
(In reply to comment #5)
> (From update of attachment 57803 [details])
> Looks good to me, thought I probably need some more time to fully understand the code.
> 
> Is Chrome doing this the name way? or what are the differences?

Thank you for taking the time to look at this.
Chrome's approach is very different than mine. They wrap the Notification object with a WebNotification object, and call the UI to handle it. They keep their queue outside of WebKit.
I was asked that notifications will work "out of the box" in QtWebKit, so I could not use the same approach.

Also, Chrome was not deleting the Notification objects until r60569 which caused an unfortunate regression (r60590), so I think they have some more work to do.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.