[webkit-dev] New runtime setting: css3TextDecoration
brunoabinader at gmail.com
Tue Aug 14 12:42:59 PDT 2012
On Tue, Aug 14, 2012 at 3:06 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:
> On Tue, Aug 14, 2012 at 11:56 AM, Julien Chaffraix
>> > * If your feature is protected by an ENABLE flag, vendors that want to
>> > ship release software can turn it off.
>> That's true, except that the original thread didn't mention any form
>> of feature flag . Nobody objected at the time and thus patches that
>> implemented part of the spec arrived on bugzilla: prefixed but not
>> protected by any flag.
> I had certainly assumed that this was done under a new build flag. If that
> were not the case, I expected relevant reviewers to r- those patches. Maybe
> this was a bad assumption to make.
I've studied CSS Region implementation and basically it contains both
compile and runtime flags: The former gets enabled by default while
the latter gets disabled, so we can still catch compile-time code
errors while not automatically exposing the feature to the web without
explicitly enabling the setting. I guess we can implement this feature
the same way?
I do have a working patch with compile flag implementation  . The
"-webkit-text-decoration-line" code works most of the time as an alias
to original "text-decoration", so it was assumed that no special build
flag was required at that time. This is fixed also in . The runtime
flag patch  works on top of , so we end up with the same
behavior as CSS region implements.
Bruno de Oliveira Abinader
More information about the webkit-dev