[Webkit-unassigned] [Bug 175259] [GTK] Implement JavaScript dialog methods of API::AutomationSessionClient
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Aug 7 22:42:25 PDT 2017
https://bugs.webkit.org/show_bug.cgi?id=175259
--- Comment #4 from Carlos Garcia Campos <cgarcia at igalia.com> ---
(In reply to Michael Catanzaro from comment #3)
> Comment on attachment 317408 [details]
> Patch
>
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=317408&action=review
>
> > Source/WebCore/platform/gtk/po/POTFILES.in:33
> > +../../../WebKit/UIProcess/API/gtk/WebKitScriptDialogGtk.cpp
>
> Do we have a script that reminds you to do this, or are you the hero of the
> translators?
Not that I know of, I guess I'm the hero of the translators, then :-D
> > Source/WebKit/ChangeLog:11
> > + files. Implement all JavaScript dialog methods of API::AutomationSessionClient in WebKitAutomationSession. For
> > + now it only works when the user doesn't override WebKitWebView::script-dialog signal and default implementation
> > + is used.
>
> Well that's a significant limitation.
It would be significant if applications actually overrode the default implementation, but I bet nobody is doing that (I don't really know, though).
> > Source/WebKit/UIProcess/API/glib/WebKitWebView.cpp:2051
> > + // FIXME: Add API to ask the user in case default implementation is not being used.
>
> What should that API look like?
I don't know, but more or less the same we are currently using internally, but using signals to ask the user about the js dialogs. Or we could add a WebKitScriptDialogManager or something like that, to not add more signals to the WebKitWebView. I don't know i haven't thought about it yet.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170808/81208533/attachment-0001.html>
More information about the webkit-unassigned
mailing list