[Webkit-unassigned] [Bug 22551] [GTK] API to start inspector for a WebView

[bugzilla-daemon at webkit.org]

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





------- Comment #16 from gns at gnome.org  2009-04-21 15:29 PDT -------
(In reply to comment #14)
> /**
>  * webkit_web_inspector_inspect:
>  * @web_inspector: a #WebKitWebInspector
>  * @inspector_panel: a string identifying the panel to show
>  *
>  * Activates the #WebKitWebInspector and switches it to the specified inspector
>  * panel. The inspector is activated with the document node of the currently
>  * focused or the main frame of the associated #WebKitWebView. If the panel
>  * string is NULL or specifies an unkown panel, no switching will occur.
>  *
>  * Since: 1.1.6
>  **/
> void webkit_web_inspector_inspect(WebKitWebInspector *web_inspector,
>                                   const gchar *inspector_panel)

That sounds good to me, but I think the verb has to be changed from inspect to
something else. The best option I could come up with up to now is 'start'; I
had considered things such as 'show', 'launch', 'present', but these have
well-defined meanings in GTK+ already, so I propose using 'start' or 'open' as
suggested by Christian, whatever works best for you.

> followed by inspect_element(web_inspector, inspector_panel, dom_element) as
> soon as the dom bindings are available.

Right =).

> If the above is ok, I'll redo the patch accordingly. Actually I started
> already, but it's going to take few nights since someone forgot to add the 12
> extra hours to a day so there's good time for feedback...

Thanks!


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