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

[bugzilla-daemon at webkit.org]

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


Laszlo Gombos <laszlo.1.gombos at nokia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |laszlo.1.gombos at nokia.com




--- Comment #12 from Laszlo Gombos <laszlo.1.gombos at nokia.com>  2010-06-08 13:45:30 PST ---
(In reply to comment #10)
> > The design in Chrome is very different than our design. They simply wrap the Notification object with a WebNotification, and pass it to the client. I suggested to make the icon loader cross platform, but John Greg told me that if I do that, Chrome would not use it.
> 
> What about the other ports? Have you talked to the Gtk guys?

I do not see a problem keeping the implementation specific to Qt initially. As Yael mentioned for technical reasons it would be difficult to share much code with Chrome, which is the only other ports at the moment that supports notifications. 

If other ports come along (e.g. Gtk) we can certainly move the implementation cross-platform, but at the moment QtWebKit is the only port where this patch can be really tested.

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