[Webkit-unassigned] [Bug 40005] [Qt] Platform plugin support for Notifications UI

[bugzilla-daemon at webkit.org]

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


Yael <yael.aharon at nokia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #58592|0                           |1
        is obsolete|                            |
  Attachment #58596|                            |review?
               Flag|                            |




--- Comment #15 from Yael <yael.aharon at nokia.com>  2010-06-13 10:56:49 PST ---
Created an attachment (id=58596)
 --> (https://bugs.webkit.org/attachment.cgi?id=58596)
Patch

Address comment #13, with the exception that I don't make a local copy of the strings (as said in comment #14).
I agree that the name NotificationIconWrapper does not reflect its purpose anymore.
Originally, I wanted it to only wrap QSystemTrayIcon, but now it does much more.
How about renaming it NotificationWrapper?
However, let's rename it in a separate patch, because this one just keeps growing :-)

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