The numbers are displayed in reverse

VERIFIED FIXED in Firefox 45

Status

()

defect
VERIFIED FIXED
4 years ago
3 years ago

People

(Reporter: over68, Assigned: jfkthame)

Tracking

({regression})

45 Branch
mozilla45
x86_64
Windows 7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 verified)

Details

Attachments

(3 attachments)

Reporter

Description

4 years ago
Steps to reproduce:

The number is displayed 81 instead of 18 in the page https://dl.dropboxusercontent.com/u/95157096/85f61cf7/lj774qbaxg.html.


Actual results:

The number is displayed incorrectly.

Screenshot https://dl.dropboxusercontent.com/u/95157096/85f61cf7/vp7gru0xdy.png
Flags: needinfo?(jfkthame)
Reporter

Updated

4 years ago
Blocks: 1157727
Reporter

Updated

4 years ago
Keywords: regression
Assignee

Comment 2

4 years ago
[Tracking Requested - why for this release]:
This is a regression that results in incorrect (misleading) display; it can actually corrupt the apparent meaning of the text. If we can't fix it reliably, we should back out bug 1157727 rather than risk shipping this.
Status: UNCONFIRMED → NEW
Has Regression Range: --- → yes
Ever confirmed: true
Flags: needinfo?(jfkthame)
Assignee

Comment 4

4 years ago
Here's a reftest based on the reduced testcase; currently fails.
Attachment #8696873 - Flags: review?(smontagu)
Assignee

Updated

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

Comment 5

4 years ago
The problem turned out to be a change that was missed when porting the updates from ICU (see source/common/ubidiimp.h) in bug 1157727. This resulted in us incorrectly considering the text to be a unidirectional RTL run at the end of ResolveExplicitLevels in this example, because DirectionFromFlags didn't recognize the digits as being LTR.
Attachment #8696874 - Flags: review?(smontagu)
Attachment #8696874 - Flags: review?(smontagu) → review+
Attachment #8696873 - Flags: review?(smontagu) → review+
Assignee

Comment 7

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/c8b058539dfd865df1e6c8b01e94992d96df879e
Bug 1231175 - Reftest for failure to recognize mixed directionality in RTL para beginning with number within embed/isolate controls. r=smontagu

https://hg.mozilla.org/integration/mozilla-inbound/rev/db10960e8ea11debb896e7dcff01d971ef09827f
Bug 1231175 - Missed change from Unicode 6.3 bidi algorithm update (ported from ICU's ubidi): include new ENL and ENR bidi pseudo-types in MASK_LTR. r=smontagu

Comment 8

4 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c8b058539dfd
https://hg.mozilla.org/mozilla-central/rev/db10960e8ea1
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Reporter

Updated

4 years ago
QA Whiteboard: [good first verify]

Comment 9

3 years ago
 [testday-20160219]  Visited website https://dl.dropboxusercontent.com/u/95157096/85f61cf7/lj774qbaxg.html . Number displayed as 18. Bug fix verified.
Marking as verified based on comment 9.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.