[webkit-dev] Adding CSS3 background-position offsets.

[Alexis Menard]

Le 10 déc. 2012 09:24, "Alexis Menard" <alexis at webkit.org> a écrit :
>
> On Tue, Dec 4, 2012 at 8:36 AM, Alexis Menard <alexis at webkit.org> wrote:
> > On Mon, Dec 3, 2012 at 5:59 PM, Dirk Schulze <dschulze at adobe.com> wrote:
> >>
> >> On Dec 3, 2012, at 12:07 PM, Benjamin Poulain <benjamin at webkit.org>
wrote:
> >>
> >>> On Mon, Dec 3, 2012 at 11:11 AM, Dirk Schulze <dschulze at adobe.com>
wrote:
> >>> Why does this feature have a flag at all? background-position with up
to 4 arguments is specified with CSS3 background and borders. There are
three major implementations with this feature. So we are just catching up.
If the future works as expected (and we should have tests that verify it),
then we should remove the flag completely.
> >>>
> >>> Sounds like good practice to me.
> >>>
> >>> Why _not_ start that feature with a flag then remove the flag when
fully ready/tested???
> >>
> >> Depends on the future. But for such a small patch, a new flag seems to
be overdone. I looked into the patch, and adding the flag caused more code,
then the actual feature (even if the computed style is missing). I believe
we should remove the flag ASAP. Flags are for bigger new features of
features that likely will change IMO. Both seems not to be the case for
background-position.
> >
> > The computed style is there see
> > http://trac.webkit.org/changeset/136378. I split the whole thing into
> > two pieces to ease the review process.
>
> So as today the feature is enable on all ports with bots (Chromium,
> Mac, GTK, Qt, EFL) and tests are doing fine. We had to rebase a test
> in ietestcenter that was buggy because it was landed with a wrong
> expected png, luckily the Chromium bots were here to tell me.
>
> I also turned on the feature for -webkit-mask which now align with the
spec [1].
>
> Anybody objects to remove the flag?
>
>

The patch is there https://bugs.webkit.org/show_bug.cgi?id=104539 up for
review. The mac ews is sick at the moment but my work environment is Mac so
it should be fine.

> [1]
http://dvcs.w3.org/hg/FXTF/raw-file/tip/masking/index.html#mask-property
>
> >
> >>
> >> Greetings,
> >> Dirk
> >>
> >>>
> >>> Benjamin
> >>> _______________________________________________
> >>> webkit-dev mailing list
> >>> webkit-dev at lists.webkit.org
> >>> http://lists.webkit.org/mailman/listinfo/webkit-dev
> >>
> >> _______________________________________________
> >> webkit-dev mailing list
> >> webkit-dev at lists.webkit.org
> >> http://lists.webkit.org/mailman/listinfo/webkit-dev
> >
> >
> >
> > --
> > Software Engineer @
> > Intel Open Source Technology Center
>
>
>
> --
> Software Engineer @
> Intel Open Source Technology Center
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20121210/586f7117/attachment.html>