Bug 87995

Summary: [Gtk] Add a configuration option for disabling unstable features in releases
Product: WebKit Reporter: Zan Dobersek <zan>
Component: WebKitGTKAssignee: Zan Dobersek <zan>
Status: RESOLVED FIXED    
Severity: Normal CC: mrobinson
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 87127    
Attachments:
Description Flags
Patch none

Description Zan Dobersek 2012-05-31 11:34:15 PDT
As discussed in https://bugs.webkit.org/show_bug.cgi?id=87127 and later on IRC, there are occasions where the feature might be supported (more or less) completely by the Gtk port but is not ready for shipping in release versions due to incompleteness or instability (spec-wise). Two features that might fall under this categorization in the feature are perhaps those guarded by the ENABLE_GAMEPAD and ENABLE_MEDIA_STREAM defines.

To achieve this a configuration option should be added - when enabled (through build-webkit or manually) the features are enabled and compiled, when disabled (the default value, when compiling a release) the features are not included in the build.
Comment 1 Zan Dobersek 2012-07-06 09:58:19 PDT
This blocking bug #87127 makes more sense.
Comment 2 Zan Dobersek 2012-07-06 11:03:11 PDT
Created attachment 151104 [details]
Patch
Comment 3 Martin Robinson 2012-07-06 11:07:56 PDT
Comment on attachment 151104 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=151104&action=review

> configure.ac:1510
> + Unstable features                                        : $enable_unstable_features

Perhaps it would be better to hide this option. I don't think we want downstream turning it on.
Comment 4 Zan Dobersek 2012-07-06 11:26:54 PDT
(In reply to comment #3)
> (From update of attachment 151104 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=151104&action=review
> 
> > configure.ac:1510
> > + Unstable features                                        : $enable_unstable_features
> 
> Perhaps it would be better to hide this option. I don't think we want downstream turning it on.

Very well, I'll remove it when landing.
Comment 5 Zan Dobersek 2012-07-06 11:54:37 PDT
Landed in http://trac.webkit.org/changeset/121988.