[Webkit-unassigned] [Bug 105631] [GTK] Add support for loading web process extensions

[bugzilla-daemon at webkit.org]

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


Gustavo Noronha (kov) <gns at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #181029|review?                     |review+
               Flag|                            |




--- Comment #13 from Gustavo Noronha (kov) <gns at gnome.org>  2013-01-09 05:06:36 PST ---
(From update of attachment 181029)
View in context: https://bugs.webkit.org/attachment.cgi?id=181029&action=review

Looks good to me, would be good to have a facility to let applications specify a private modules path so that we don't end up with a lot of modules that won't be used loaded unnecessarily, or with applications relying on others' modules. Other than that the API looks great to me, too.

> Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitWebExtension.cpp:117
> + *
> + * Get the web page of the given @page_id.
> + *
> + * Returns: the #WebKitWebPage for the given @page_id, or %NULL if the
> + *    identifier doesn't correspond to an exsiting web page.

Be useful to state the WebExtension owns the object that is returned and annotate transfer none.

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