Rename isSpace to treatAsSpace. Move it and the rounding hack function into
[WebKit-https.git] / WebCore / ChangeLog
1 2006-05-10  David Hyatt  <hyatt@apple.com>
2
3         Rename isSpace to treatAsSpace.  Move it and the rounding hack function into
4         Font and make them static methods (inlined in the header).  Make the rounding
5         hack character table a static member as well.  Remove the redundant space/rounding
6         functions from FontData.mm.
7
8         Reviewed by darin
9
10         * platform/Font.cpp:
11         (WebCore::):
12         (WebCore::m_finalRoundingWidth):
13         (WebCore::WidthIterator::advance):
14         * platform/Font.h:
15         (WebCore::Font::treatAsSpace):
16         (WebCore::Font::isRoundingHackCharacter):
17         * platform/mac/FontData.mm:
18         (WebCore::overrideLayoutOperation):
19         (WebCore::createATSULayoutParameters):
20         (WebCore::initializeWidthIterator):
21         (WebCore::advanceWidthIterator):
22         * platform/mac/FontMac.mm:
23         (WebCore::overrideLayoutOperation):
24         (WebCore::ATSULayoutParameters::initialize):
25
26 2006-05-10  Darin Adler  <darin@apple.com>
27
28         * WebCore: Removed an extra WebCore subtree that somehow got checked in.
29
30 2006-05-10  David Carson <dacarson@gmail.com>
31
32         - Fix for bug 8833. Removed inline keyword from functions declared
33         in .mm and .cpp files. inline function code needs to reside in the
34         header file for the linker to find the code. 
35         http://www.parashift.com/c++-faq-lite/inline-functions.html#faq-9.7
36
37         Reviewed by Darin, landed by Timothy.
38
39         * platform/Font.cpp:
40         (WebCore::isSpace):
41         * platform/mac/FontData.mm:
42         (WebCore::widthForGlyph):
43         (WebCore::isRoundingHackCharacter):
44         (WebCore::glyphForCharacter):
45
46 2006-05-10  Mitz Pettel  <opendarwin.org@mitzpettel.com>
47
48         Reviewed and landed by Anders.
49
50         - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=8830
51           REGRESSION: Words render on top of each other when fallback font is used
52
53         Already covered by several pixel tests in fast/text
54
55         * platform/Font.cpp:
56         (WebCore::Font::drawSimpleText):
57
58 2006-05-10  David Hyatt  <hyatt@apple.com>
59
60         Fix for bug 8809, lift the Mac-specific WebTextRenderer code for
61         drawing and measuring of runs up into the Font class.  The fast code
62         path is in Font.cpp and is designed to be mostly cross-platform.  The
63         slow code path is in FontMac.cpp and is Mac-specific.
64
65         Reviewed by darin
66
67         * WebCore.xcodeproj/project.pbxproj:
68         * platform/Font.cpp:
69         (WebCore::isSpace):
70         (WebCore::):
71         (WebCore::isRoundingHackCharacter):
72         (WebCore::m_finalRoundingWidth):
73         (WebCore::WidthIterator::advance):
74         (WebCore::WidthIterator::normalizeVoicingMarks):
75         (WebCore::Font::primaryFont):
76         (WebCore::Font::setAlwaysUseComplexPath):
77         (WebCore::Font::canUseGlyphCache):
78         (WebCore::Font::drawSimpleText):
79         (WebCore::Font::drawText):
80         (WebCore::Font::floatWidth):
81         (WebCore::Font::floatWidthForSimpleText):
82         * platform/Font.h:
83         * platform/FontData.h:
84         (WebCore::FontData::platformData):
85         * platform/GlyphBuffer.h: Added.
86         (WebCore::GlyphBuffer::GlyphBuffer):
87         (WebCore::GlyphBuffer::isEmpty):
88         (WebCore::GlyphBuffer::size):
89         (WebCore::GlyphBuffer::glyphs):
90         (WebCore::GlyphBuffer::advances):
91         (WebCore::GlyphBuffer::fontDataAt):
92         (WebCore::GlyphBuffer::swap):
93         (WebCore::GlyphBuffer::glyphAt):
94         (WebCore::GlyphBuffer::advanceAt):
95         (WebCore::GlyphBuffer::add):
96         * platform/mac/FontData.mm:
97         (WebCore::isSpace):
98         (WebCore::isRoundingHackCharacter):
99         (WebCore::FontData::widthForGlyph):
100         (WebCore::m_ATSUMirrors):
101         (WebCore::FontData::xHeight):
102         (WebCore::FontData::smallCapsFontData):
103         (WebCore::findSubstituteFont):
104         (WebCore::rendererForAlternateFont):
105         (WebCore::findSubstituteRenderer):
106         (WebCore::FontData::findSubstituteFontData):
107         (WebCore::computeWidthForSpace):
108         (WebCore::FontData::updateGlyphMapEntry):
109         (WebCore::extendGlyphMap):
110         (WebCore::extendWidthMap):
111         (WebCore::createATSULayoutParameters):
112         (WebCore::FontData::glyphForCharacter):
113         (WebCore::advanceWidthIterator):
114         (WebCore::shouldUseATSU):
115         * platform/mac/FontMac.mm:
116         (WebCore::ATSULayoutParameters::m_padPerSpace):
117         (WebCore::addDirectionalOverride):
118         (WebCore::initializeATSUStyle):
119         (WebCore::overrideLayoutOperation):
120         (WebCore::ATSULayoutParameters::initialize):
121         (WebCore::disposeATSULayoutParameters):
122         (WebCore::Font::drawComplexText):
123         (WebCore::Font::floatWidthForComplexText):
124         (WebCore::Font::drawGlyphs):
125         * platform/mac/WebCoreTextRenderer.mm:
126         (WebCoreSetAlwaysUseATSU):
127
128 2006-05-10  Darin Adler  <darin@apple.com>
129
130         - another try at fixing the Windows build
131
132         * platform/win/FontWin.cpp: (WebCore::getFontData): Use characters()
133         instead of unicode().
134
135 == Rolled over to ChangeLog-2006-05-10 ==