[Webkit-unassigned] [Bug 83523] New: REGRESSION(r102190) [chromium] text rendering (font and font size) in some Arabic/Persian page is wrong when using certain fonts

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=83523

           Summary: REGRESSION(r102190) [chromium] text rendering (font
                    and font size) in some Arabic/Persian page is wrong
                    when using certain fonts
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: InChromiumBugs
          Severity: Normal
          Priority: P2
         Component: Layout and Rendering
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: xji at chromium.org


Created an attachment (id=136334)
 --> (https://bugs.webkit.org/attachment.cgi?id=136334&action=review)
test case

related chromium bug http://code.google.com/p/chromium/issues/detail?id=121754.

Looks like reverse r102190 fixes the problem.

As to the issue 73806 (https://bugs.webkit.org/show_bug.cgi?id=73806) which r102190 fixes, should the correct fix be the following?

Index: UniscribeHelper.cpp
===================================================================
--- UniscribeHelper.cpp (revision 108619)
+++ UniscribeHelper.cpp (working copy)
@@ -902,6 +921,7 @@
                 // the difference in the width of the glyph.
                 shaping.m_advance[glyphIndex] -= diff;
                 shaping.m_abc.abcB -= diff;
+                shaping.m_glyphs[glyphIndex] = shaping.m_spaceGlyph;
                 continue;
             }

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.