[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





--- Comment #14 from Kenneth Rohde Christiansen <kenneth at webkit.org>  2010-06-13 07:08:14 PST ---

> im a bit confused about this code. When you create the IconWrapper, why can't you already fill it with the title, message etc? 

What is important there is that we do not copy data unnecessarily, so my comment might be invalid. Cutting down on indirection if possible, is a good thing though.

I think the Platform Plugin API is nice now, so later we can fix the implementation if needed.

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