Use the C++ classes instead.
Created attachment 164149 [details] Patch
Created attachment 166484 [details] Updated patch to apply on current git master A patch in trunk broke this one
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Comment on attachment 166484 [details] Updated patch to apply on current git master View in context: https://bugs.webkit.org/attachment.cgi?id=166484&action=review Looks good to me! > Source/WebKit2/UIProcess/API/gtk/WebKitSettings.cpp:956 > + * WebKitSettings:media-playback-allows-inline: Haha... good catch, "vigilante"!
Created attachment 168891 [details] Updated patch to build on current git master Updated after smooth-scrolling setting
Comment on attachment 168891 [details] Updated patch to build on current git master View in context: https://bugs.webkit.org/attachment.cgi?id=168891&action=review > Source/WebKit2/UIProcess/API/gtk/WebKitSettings.cpp:-1105 > - ASSERT(!priv->userAgent.isNull()); Shouldn't you keep the ASSERT?
Comment on attachment 168891 [details] Updated patch to build on current git master View in context: https://bugs.webkit.org/attachment.cgi?id=168891&action=review >> Source/WebKit2/UIProcess/API/gtk/WebKitSettings.cpp:-1105 >> - ASSERT(!priv->userAgent.isNull()); > > Shouldn't you keep the ASSERT? I think it's impossible to happen, user-agent is a construct property, and when passing NULL it's initialized to the default user agent, so at this point, if we have a valid a WebKitSettings object, userAgent can't be NULL.
Committed r131750: <http://trac.webkit.org/changeset/131750>