LayoutTests/ChangeLog

2010-04-21  Ojan Vafai  <ojan@chromium.org>

        Reviewed by NOBODY (OOPS!).

        http://trac.webkit.org/changeset/57215 caused perf/memory regressions

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

        #if out the canUseGlyphCache Changes from r57215 as they caused a 

        8% perf regression on Chromium's international page load tests so that

        the perf regression can be fixed properly without being left in the tree.

        * platform/mac/fast/repaint/stacked-diacritics-expected.checksum:

        * platform/mac/fast/repaint/stacked-diacritics-expected.png:

2010-04-28  Xiaomei Ji  <xji@chromium.org>

        Reviewed by Dimitri Glazkov

LayoutTests/platform/mac/fast/repaint/stacked-diacritics-expected.checksum

1 61cab097914525fb7f7a07989330012e

21\ No newline at end of file

2214a62226351798b782ad9c839ab900d

33\ No newline at end of file

LayoutTests/platform/mac/fast/repaint/stacked-diacritics-expected.png

INVALID: Image lacks a checksum. This will fail with a MISSING error in run-webkit-tests. Always generate new png files using run-webkit-tests.

WebCore/ChangeLog

2010-04-21  Ojan Vafai  <ojan@chromium.org>

        Reviewed by NOBODY (OOPS!).

        http://trac.webkit.org/changeset/57215 caused perf/memory regressions

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

        #if out the canUseGlyphCache Changes from r57215 as they caused a 

        8% perf regression on Chromium's international page load tests so that

        the perf regression can be fixed properly without being left in the tree.

        * platform/graphics/FontFastPath.cpp:

        (WebCore::Font::canUseGlyphCache):

2010-04-28  Pavel Feldman  <pfeldman@chromium.org>

        Reviewed by Timothy Hatcher.

WebCore/platform/graphics/FontFastPath.cpp

#include <wtf/MathExtras.h>

#include <wtf/unicode/Unicode.h>

#ifndef ROMAN_AND_GREEK_DIACRITICS_CAN_USE_GLYPH_CACHE

#define ROMAN_AND_GREEK_DIACRITICS_CAN_USE_GLYPH_CACHE 1

#endif

using namespace WTF;

using namespace Unicode;

@@bool Font::canUseGlyphCache(const TextRun& run) const

        if (c <= 0x194F)

            return false;

241#if !ROMAN_AND_GREEK_DIACRITICS_CAN_USE_GLYPH_CACHE

237242 // FIXME: we should not use complex text path for these characters.

238

        if (c < 0x1E00)     // U+1E00 through U+2000 characters with diacritics and stacked diacritics

            continue;

        if (c <= 0x2000)

            return false;

247#endif

        if (c < 0x20D0)     // U+20D0 through U+20FF Combining marks for symbols

            continue;