Changeset 113951 in webkit


Ignore:
Timestamp:
Apr 12, 2012 1:49:04 AM (12 years ago)
Author:
bashi@chromium.org
Message:

[Chromium] The size of glyphStorage should be passed to substituteWithVerticalGlyphs()
https://bugs.webkit.org/show_bug.cgi?id=83751

Reviewed by Kent Tamura.

The last argument of substituteWithVerticalGlyphs() should be
the size of |glyphStorage|.

No new tests because we can't test the fix by layout tests.
I confirmed the fix with ASAN.

  • platform/graphics/skia/GlyphPageTreeNodeSkia.cpp:

(WebCore::GlyphPage::fill):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r113950 r113951  
     12012-04-12  Kenichi Ishibashi  <bashi@chromium.org>
     2
     3        [Chromium] The size of glyphStorage should be passed to substituteWithVerticalGlyphs()
     4        https://bugs.webkit.org/show_bug.cgi?id=83751
     5
     6        Reviewed by Kent Tamura.
     7
     8        The last argument of substituteWithVerticalGlyphs() should be
     9        the size of |glyphStorage|.
     10
     11        No new tests because we can't test the fix by layout tests.
     12        I confirmed the fix with ASAN.
     13
     14        * platform/graphics/skia/GlyphPageTreeNodeSkia.cpp:
     15        (WebCore::GlyphPage::fill):
     16
    1172012-04-12  Jason Liu  <jason.liu@torchmobile.com.cn>
    218
  • trunk/Source/WebCore/platform/graphics/skia/GlyphPageTreeNodeSkia.cpp

    r103742 r113951  
    105105        }
    106106        if (lookVariants)
    107             substituteWithVerticalGlyphs(fontData, glyphs, bufferLength);
     107            substituteWithVerticalGlyphs(fontData, glyphs, length);
    108108    }
    109109
Note: See TracChangeset for help on using the changeset viewer.