[Webkit-unassigned] [Bug 96171] [Qt] Implement WebNotification in WebKit2 port

[bugzilla-daemon at webkit.org]

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


José Dapena Paz <jdapena at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
         AssignedTo|webkit-unassigned at lists.web |jdapena at igalia.com
                   |kit.org                     |
                 CC|                            |jdapena at igalia.com
     Ever Confirmed|0                           |1




--- Comment #6 from José Dapena Paz <jdapena at igalia.com>  2013-01-16 02:56:59 PST ---
(In reply to comment #4)

>     2) QSystemTrayIcon is part of QtWidgets, which we choose not to use in the WebKit2 API.
>     3) Conceptually I'm not sure if QSystemTrayIcon is the right choice in a QML based environment. I'd much rather prefer an API that permits convenient delegation to the application or platform layer, given that the notifications are most likely use-case specific.

I am working now on a new implementation of this. We want to avoid dependency on QtWidgets, and, as QSystemTrayIcon doesn't seem to move out of it any time soon, we need a different approach.

My new approach is exposing it to application level. We'd have QWebNotification objects with methods click, close, show, to allow app to notify webkit about the changes in current notification status. My main doubt is how to expose the notifications the first time to application.

My initial idea was adding signals "showNotification" and "cancelNotification" to QQuickWebView, following an approach similar to Gtk and EFL WK1 port implementation proposals. These signals would have as parameter the QWebNotification object, and indicate the life cycle of it (app would be able to handle notifications between show and cancel). The problem is, on WebKit2 there's no reference to PageUIClient on the notification provider. We only have a WKPageRef, so I don't have a way to associate the notification to a QQuickWebView.

So my current idea is somehow expose the notifications using a global object (i.e. signals of the webkit shared object I am proposing for https://bugs.webkit.org/show_bug.cgi?id=105034 ).

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