[Webkit-unassigned] [Bug 105720] [GTK] Fix build failure and some nits

[bugzilla-daemon at webkit.org]

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





--- Comment #5 from Zan Dobersek <zandobersek at gmail.com>  2012-12-25 09:22:37 PST ---
(From update of attachment 180673)
View in context: https://bugs.webkit.org/attachment.cgi?id=180673&action=review

The GTK port is currently in the process of cleaning up the various configuration options located in configure.ac which are not really needed. In short form, if the feature (guarded by any feature define) does not introduce external dependencies, there shouldn't be a corresponding configuration flag available. An exception to this is the SVG feature, the reason for the configuration flag being that the SVG code takes notable time to compile so there should be a way of disabling it.

> Source/WebKit/gtk/webkit/webkitwebviewprivate.h:33
> +#include "ResourceResponse.h"

In what circumstances does build fail without this being added? What feature is enabled/disabled?
If it is of valid cause, please proceed with pushing to get this part of the patch landed.

> configure.ac:662
> +AC_ARG_ENABLE(video_track,

There's no need for this configuration option as it doesn't introduce any additional dependencies.

To prevent compilation failure, changes should be made to CodeGeneratorObject.pm. I've checked the build failures and created a patch for that, I'll upload it up for a review in another bug.

> configure.ac:686
> +AC_ARG_ENABLE(css_transforms_animations_transitions_unprefixed,

This configuration option is unnecessary as well, it was only added due to people adding new features being uninformed about the new configuration flag criteria on the GTK port (which is my fault, really).

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