text-transform doesn't work on non-BMP characters (plane 1 characters, Deseret)

RESOLVED FIXED in mozilla14

Status

()

Core
Layout: Text
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: dbaron, Assigned: jfkthame)

Tracking

(Blocks: 1 bug, {css1, css2})

Trunk
mozilla14
css1, css2
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
text-transform doesn't work on non-BMP characters (such as Deseret) because  nsCaseTransformTextRunFactory::RebuildTextRun and the code it uses operates on 16-bit code units.

This makes us fail:
http://test.csswg.org/suites/css2.1/20101001/html4/text-transform-bicameral-021.htm
http://test.csswg.org/suites/css2.1/20101001/html4/text-transform-bicameral-022.htm
http://test.csswg.org/suites/css2.1/20101001/xhtml1/text-transform-bicameral-021.xht
http://test.csswg.org/suites/css2.1/20101001/xhtml1/text-transform-bicameral-022.xht
Whiteboard: DUPEME
(Reporter)

Updated

7 years ago
Blocks: 605520
(Assignee)

Comment 1

5 years ago
Created attachment 602828 [details] [diff] [review]
patch, handle surrogates in text-run case transformation

This depends on bug 210501 to provide the support for case-mapping of non-BMP character codes; once that infrastructure is done, we can fix text-transform to handle Deseret properly. :) (And any future casing alphabet that may get added to Plane 1...)
Attachment #602828 - Flags: review?(smontagu)
(Assignee)

Updated

5 years ago
Depends on: 210501
Attachment #602828 - Flags: review?(smontagu) → review+
(Assignee)

Comment 2

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/6913a9ff86e1
Assignee: nobody → jfkthame
Whiteboard: DUPEME
Target Milestone: --- → mozilla14
https://hg.mozilla.org/mozilla-central/rev/6913a9ff86e1
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Depends on: 736210
You need to log in before you can comment on or make changes to this bug.