New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 641937 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Unicode dotted circle character does not combine properly with combining accents

Reported by tszynal...@antimoon.com, Aug 29 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:48.0) Gecko/20100101 Firefox/48.0

Example URL:
https://jsfiddle.net/tszynalski/7jk2y633/2/

Steps to reproduce the problem:
1. Go to https://jsfiddle.net/tszynalski/7jk2y633/2/
2. Press Run

You can also see the bug here:
https://en.wikipedia.org/wiki/Dotted_circle

What is the expected behavior?

What went wrong?
The accents are not properly placed over the dotted circle. It's the same for a number of different fonts.

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? N/A 

Does this work in other browsers? Yes 

Chrome version: 52.0.2743.116 m  Channel: n/a
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: Shockwave Flash 22.0 r0

This is kind of important as the dotted circle character is used to illustrate the effect of combining diacritical marks.
 
Components: -Blink Blink>Fonts
Cc: behdad@chromium.org e...@chromium.org rnimmagadda@chromium.org
Labels: -Type-Bug M-53 OS-Linux OS-Mac Type-Bug-Regression
Owner: drott@chromium.org
Status: Assigned (was: Unconfirmed)
====================================

Good Build:

50.0.2633.0    Base Position: 371964


Bad Build:

50.0.2645.0    Base Position: 374251

=====================================

Able to repro this issue on Windows 7, MAC (10.11.6) & Ubuntu Trusty (14.04) for the Google Chrome Stable Version - 52.0.2743.116	

This is a regression issue broken in M50, below mentioned is the bisect info:

CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/12827b68174c363351b05a8e7f7c29e1d9c0c01d..38fecdf10f76b2774fa844336dddd2a97b68b5f6

Suspecting Commit: 8d0aeb2dced892bda385668da1fae29236e858b5

Review URL: https://codereview.chromium.org/1613903003

@drott: Could you please look into the issue, and if it has nothing to do with your changes and if possible please do assign it to the concerned owner.

Thank you.

Comment 3 by drott@chromium.org, Sep 8 2016

Thanks for the report, fix candidate up in https://codereview.chromium.org/2321043002

Project Member

Comment 4 by bugdroid1@chromium.org, Sep 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3ec36e0154e67ab8eda029ae5a234d581663e226

commit 3ec36e0154e67ab8eda029ae5a234d581663e226
Author: drott <drott@chromium.org>
Date: Thu Sep 08 15:14:16 2016

Disable Math and Symbols segmentation, fix dotted circle sequence

We currently do not use the Math and Symbols category for font
selection, hence disabling it, since it was also breaking sequences of
dotted circle plus combining marks which are frequently used to explain
combining marks.

BUG= 641937 
R=eae,kojii

Review-Url: https://codereview.chromium.org/2321043002
Cr-Commit-Position: refs/heads/master@{#417298}

[modify] https://crrev.com/3ec36e0154e67ab8eda029ae5a234d581663e226/third_party/WebKit/Source/platform/fonts/FontFallbackPriority.cpp
[modify] https://crrev.com/3ec36e0154e67ab8eda029ae5a234d581663e226/third_party/WebKit/Source/platform/fonts/FontFallbackPriority.h
[modify] https://crrev.com/3ec36e0154e67ab8eda029ae5a234d581663e226/third_party/WebKit/Source/platform/fonts/OrientationIteratorTest.cpp
[modify] https://crrev.com/3ec36e0154e67ab8eda029ae5a234d581663e226/third_party/WebKit/Source/platform/fonts/ScriptRunIteratorTest.cpp
[modify] https://crrev.com/3ec36e0154e67ab8eda029ae5a234d581663e226/third_party/WebKit/Source/platform/fonts/SymbolsIterator.cpp
[modify] https://crrev.com/3ec36e0154e67ab8eda029ae5a234d581663e226/third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp
[modify] https://crrev.com/3ec36e0154e67ab8eda029ae5a234d581663e226/third_party/WebKit/Source/platform/fonts/shaping/RunSegmenterTest.cpp

Comment 5 by e...@chromium.org, Sep 8 2016

Status: Fixed (was: Assigned)

Sign in to add a comment