Bug 14726 – [gtk] API design. Mapping the WebView delegates to signals.

• Component: New Bugs
• Status: RESOLVED FIXED

[Holger Freyther]

It is important to design a suitable API for the WebKitGtk port. The primary use case should be browsing and displaying content. The secondary use case will be editing.

The API should follow WebView of the Mac/Windows port both in design and naming of functions. We are free to name the methods slightly different and insert as many '_' as we want.

WebView is using delegates. For the WebKitGtk port these will be mapped to signals. For the informal signals the only decision we have to make is if we want to allow having default handlers in the class of WebKitGtkPage and WebKitGtkFrame.

For delegates like the WebPolicyDelegate we should not use signals with return values but make the decisionListener some kind of GType (boxed,GObject...) that gets passed to the handlers as part of the signal. The first one calling the decisionListener makes the decision.

For delegates like the WebFrameLoadDelegate (responsible for assigning ids) we should consider using return values or pass the 'result' as pointer in the signal. The issue to be discussed is who should be able to stop the emission of the signal, who should be able to make the decision. The first one returning a result, the last one setting a result in the struct?

[Holger Freyther]

Rename the symbols from webkit_gtk to web_ and the files from webkitgtk to web (priority :fix it!!).

[Holger Freyther]

Add the Gtk keyword, even to the curl bugs.

[Christian Dywan]

> Rename the symbols from webkit_gtk to web_ and the files from webkitgtk to web

Please do *not* change it to web_. That would be too generic and therefore unfair against other projects. Think of web_page_new for instance. It might very easily clash with other names.

Instead my suggestion for the scheme would be either webkit_foo or gtk_webkit_foo. I tend to be in favor of gtk_webkit_foo because WebKit is a cross-platform toolkit and omitting the gtk suggests a gtk-specific toolkit imho.

[Alp Toker]

*** Bug 15357 has been marked as a duplicate of this bug. ***

[Alp Toker]

web_ is not a popular option with the developers I've spoken with, due to namespace pollution. wk_ was also suggested but webkit_ seems to be the most sensible option. I wonder now whether the CamelCased form of this namespace should be Webkit or WebKit (ie. WebkitPage vs. WebKitPage)..

[Alp Toker]

Created attachment 16520 [details]
Rename the Gtk+ port API namespace

[Adam Roben (:aroben)]

Comment on attachment 16520 [details]
Rename the Gtk+ port API namespace

If this is truly just a rename as claimed, rs=me. We should rename the files at some point as well (webkitgtkpage.cpp -> webkitpage.cpp)

[Alp Toker]

Fixed in r26022