[Webkit-unassigned] [Bug 41413] [Qt] QtWebKit needs public API for Notifications.

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Yael <yael.aharon at nokia.com>  2010-06-30 11:25:35 PST ---
(In reply to comment #2)
> (From update of attachment 60128 [details])
> WebKit/qt/Api/qwebpage.h:204
>  +      enum UserPermissionType {
> Isn't that more domain than type? PermissionDomain ?
> 
> WebKit/qt/Api/qwebpage.h:198
>  +      enum UserPermission {
> PermissionPolicy ?
> 
> WebKit/qt/Api/qwebpage.h:303
>  +      void setUserPermission(UserPermissionType, const QString&, UserPermission);
> void setUserPermission(PermissionDomain domain, const QString&, PermissionPolicy policy) ?
> 
> WebKit/qt/Api/qwebpage.h:199
>  +          UserPermissionAllowed,
> maybe Granded. You grand permission I believe.
> 
> 
> Are all these other changes really related to the public API?

Kenneth, thanks for reviewing the API proposal. Unfortunately, all the changes are related to the new API, including WebKit/chromium changes. The current NotificationPresenter interface does not give us enough information to know which page initiated the notification, so if we wanted to show a UI, we would not know which tab/window should show that UI.
In Chromium this is not a problem because they know which process initiated the IPC.

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