[Webkit-unassigned] [Bug 137116] [GTK] Add selection-changed signal to the WebKit2 API

[bugzilla-daemon at webkit.org]

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

--- Comment #19 from Tomas Popela <tpopela at redhat.com> ---
(In reply to comment #18)
> > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitWebEditor.cpp:115
> > +/**
> > + * webkit_web_editor_get_page:
> > + * @editor: a #WebKitWebEditor
> > + *
> > + * Gets the #WebKitWebPage that is associated with #WebKitWebEditor.
> > + *
> > + * Returns: (transfer none): the #WebKitWebPage that can be used to access
> > + * the #WebKitDOMDocument currently loaded into it.
> 
> If this is mainly useful for accessing the DOM, maybe it makes more sense to
> have webkit_web_editor_get_dom_document?
> 

As you wrote in #c12:

> the page associated to the editor, that can be used for anything else apart from accessing the DOM

I would not do a special API just for accessing the DOM as for me it looks like some unnecessary and unwanted duplication. You are also closing the doors for accessing information that are in the WebKitWebPage (we really don't know what we will add there in the future) and I don't think we want to have getters for every interesting information in WebKitWebEditor's POV.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150714/3e11cd76/attachment.html>