[Webkit-unassigned] [Bug 108482] [WebCore][Notifications] Do not assume the page has a NotificationClient

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 7 07:54:38 PST 2013


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





--- Comment #8 from Jon Lee <jonlee at apple.com>  2013-02-07 07:56:46 PST ---
(In reply to comment #7)
> (In reply to comment #6)
> > Actually, doesn't it make more sense to not call provideNotification(), and check for the existence of a controller?
> 
> Actually the caller of Notification::requestPermission() seems to be JSNotification::requestPermission(). Not sure I understand what you mean?

If you don't have a notification client, then you shouldn't be adding a NotificationController as a supplement to the Page. So I think you should be able to check for a null NotificationController, rather than  checking for a null NotificationClient.

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


More information about the webkit-unassigned mailing list