"ASSERTION: Getting CharProps for codepoint outside Unicode range" with small-caps

RESOLVED FIXED in mozilla33

Status

()

Core
Graphics: Text
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Jesse Ruderman, Assigned: jfkthame)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
mozilla33
x86_64
Mac OS X
assertion, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 1 obsolete attachment)

(Reporter)

Description

4 years ago
Created attachment 8450711 [details]
testcase that asserts

###!!! ASSERTION: Getting CharProps for codepoint outside Unicode range: 'Not Reached', file intl/unicharutil/util/nsUnicodeProperties.cpp, line 51

The codepoint passed to GetCharProps2 is indeed outside the Unicode range: it is 4294967295.

Also affects <canvas> text methods.
(Reporter)

Comment 1

4 years ago
Created attachment 8450712 [details]
stack
(Reporter)

Comment 2

4 years ago
Created attachment 8450713 [details]
testcase that looks wrong
(Reporter)

Comment 3

4 years ago
Somehow the second testcase renders as X∞≤ (X U+221E U+2264).

I have no idea how U+E0131 turns into U+221E U+2264.
(Assignee)

Comment 4

4 years ago
Created attachment 8450825 [details] [diff] [review]
update index to step past the second half of a surrogate pair.
Attachment #8450825 - Flags: review?(roc)
(Assignee)

Updated

4 years ago
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
(Assignee)

Comment 5

4 years ago
Comment on attachment 8450825 [details] [diff] [review]
update index to step past the second half of a surrogate pair.

I don't think this is quite right, actually; cancelling r? until I look more carefully.
Attachment #8450825 - Flags: review?(roc)
(Assignee)

Comment 6

4 years ago
Created attachment 8451175 [details] [diff] [review]
update index to step past the second half of a surrogate pair.
Attachment #8451175 - Flags: review?(roc)
(Assignee)

Updated

4 years ago
Attachment #8450825 - Attachment is obsolete: true
Comment on attachment 8451175 [details] [diff] [review]
update index to step past the second half of a surrogate pair.

Review of attachment 8451175 [details] [diff] [review]:
-----------------------------------------------------------------

don't forget the test
Attachment #8451175 - Flags: review?(roc) → review+
(Assignee)

Comment 8

4 years ago
Pushed to inbound (including the asserting testcase):
https://hg.mozilla.org/integration/mozilla-inbound/rev/eed019320764
Target Milestone: --- → mozilla33
https://hg.mozilla.org/mozilla-central/rev/eed019320764
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.