[webkit-dev] Shape-inside removal, bug #130698

[Zoltan Horvath]

Hi there,

The patch is landed in http://trac.webkit.org/changeset/166301
I already did the first step to simplify our geometry codebase in
http://trac.webkit.org/changeset/166316

Thumbs up for our shiny shape-outside! :-)

Cheers,
Zoltan


On Tue, Mar 25, 2014 at 12:59 PM, Zoltan Horvath <zoltan at webkit.org> wrote:

> Hi there,
>
> CSS Shapes Level 1 (CR [1]) only contains shape-outside. We are focusing our effort on finalizing the implementation of shape-outside, so it's worth to remove shape-inside code at this point for now.
>
> A list of reasons for the removal:
>
> - Shape-inside is only part of Shapes Level 2, which needs to be improved on some topics.
> - Shape-inside is lack of new shapes support (e.g. inset).
> - Deprecated shapes (r165472) are removed from the code (e.g. rectangle), which affects shape-inside.
> - The current shape-inside code spreads across the layout code.
> - The current shape-inside implementation is experimental in some areas, and the partially implemented code can have security implications.
> - Removal of shape-inside opens possibilities for code complexity and performance optimizations for shape-outside. (e.g. simpler geometry code)
>
> I've got the patch reviewed in bug #130698 [2] ready for landing. If there are no objections, I'll land the patch tomorrow morning.
>
> Cheers,
> Zoltan
>
> [1] http://www.w3.org/TR/2014/CR-css-shapes-1-20140320/
> [2] https://bugs.webkit.org/show_bug.cgi?id=130698
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-dev/attachments/20140326/59b4a5d9/attachment.html>