Bug 48085 – Make basic vertical text painting work.

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Dave Hyatt]

Make basic vertical text painting work.  This includes the text itself, underlines, overlines, line-throughs and shadows.

Selection would draw correctly if hit testing worked well or if invalidation worked well. :)

[Dave Hyatt]

Created attachment 71469 [details]
Patch

[Darin Adler]

Comment on attachment 71469 [details]
Patch

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

> WebCore/WebCore.xcodeproj/project.pbxproj:-21155
> -			developmentRegion = English;

You need to update your Xcode with Software Update. The newer version doesn’t remove this, and you should not check in this project change.

> WebCore/rendering/InlineFlowBox.cpp:-708
> -            paintTextDecorations(paintInfo, tx, ty, false);

How does this work now? Is there other code that always paints text decorations now that used to do it only conditionally?

> WebCore/rendering/InlineTextBox.cpp:433
> +        context->rotate(deg2rad(90.));

We normally would just use 90 here instead of "90." but maybe that will cause overloading problems with the deg2rad function. Also, this will call deg2rad at runtime -- maybe we should include a constant you can use for this. Maybe a helper function to rotate 90 degrees would be cleaner?

[Dave Hyatt]

(In reply to comment #2)
>
> > WebCore/rendering/InlineFlowBox.cpp:-708
> > -            paintTextDecorations(paintInfo, tx, ty, false);
> 
> How does this work now? Is there other code that always paints text decorations now that used to do it only conditionally?
> 

We have two complete text decoration drawing code paths.  One is for quirks mode and one is for strict mode.  CSS2.1 has defined a third model that is not compatible with either of our models (yay).  However our quirks mode model is much closer to what CSS2.1 wants.  In addition CSS2.1 states that "older UAs that predate CSS2.1 are still conformant" if we implement text-decoration the way our quirks mode model does.

This patch is removing the strict mode text decoration code path and just making the quirks mode one the only path we use.

[Dave Hyatt]

Created attachment 71489 [details]
Patch that adds new results from Takano's change.

[Dave Hyatt]

Fixed in r70263.

[Dave Hyatt]

Created attachment 71493 [details]
New ugly results from Takano's and my patch combining.

Patch

[Eric Seidel (no email)]

This broke snow leopard.

[Ryosuke Niwa]

(In reply to comment #7)
> This broke snow leopard.

Fixed in http://trac.webkit.org/changeset/70281.  Please verify the fix.