[Webkit-unassigned] [Bug 113882] New: REGRESSION(130231): Leaves m_glyphToCharacterIndexes uninitialized

[bugzilla-daemon at webkit.org]

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

           Summary: REGRESSION(130231): Leaves m_glyphToCharacterIndexes
                    uninitialized
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Platform
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: dominik.rottsches at intel.com
                CC: tony at chromium.org, bashi at chromium.org,
                    glenn at skynav.com


As commented in https://bugs.webkit.org/show_bug.cgi?id=97993#c13
the patch from r130231 removes

-    hb_glyph_info_t* infos = hb_buffer_get_glyph_infos(harfBuzzBuffer, 0);
-    for (unsigned i = 0; i < m_numGlyphs; ++i)
-        m_glyphToCharacterIndexes[i] = infos[i].cluster;

in HarfBuzzShaper::HarfBuzzRun::applyShapeResult.

And I can't see any replacement for that in r130231. AFAICS, with that, m_glyphToCharacterIndexes becomes uninitialized.

Unfortunately, I don't have a test to prove it yet. But I am suspecting it breaks selection in certain cases.

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