Bug 84484

Summary:	[Qt] Refactor notification icon loading out of notification.cpp
Product:	WebKit	Reporter:	Jon Lee <jonlee>
Component:	New Bugs	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED DUPLICATE
Severity:	Normal	CC:	jianli, yael
Priority:	P2
Version:	528+ (Nightly build)
Hardware:	Mac
OS:	OS X 10.7

Jon Lee

Reported 2012-04-20 13:03:57 PDT

At this point the loading of the notification icon is happening only in the Qt port, and the api needed to support this is making the notifications object a more complex. Since this is an optional feature, it would be best to factor this out into its own NotificationIconLoader class. This grew out of reviews of bug 80477.

Attachments
Add attachment proposed patch, testcase, etc.

Jon Lee

Comment 1 2012-04-20 13:17:04 PDT

Didn't see the already filed bug. Oops. Duping. *** This bug has been marked as a duplicate of bug 80702 ***

Yael

Comment 2 2012-04-20 16:36:00 PDT

*** This bug has been marked as a duplicate of bug 80700 ***

Note You need to log in before you can comment on or make changes to this bug.