|
Remove Simple Text Path
Concluding a 3 year effort which unified the shaping logic across all
platforms, improved the performance of the complex text path through a
word cache, further improved the correctness of the complex text path
and removed all remaining dependencies on simple text path constructs
from SimpleFontData, SVG text and InlineTextBox, fixing tricky memory
regressions on Android we are now ready to remove the simple text code
path.
This means we now only have one code path for text processing removing a
class of bugs related to discrepancies between the code paths and get
rid of the legacy maintenance burden that was the simple text code
path. All our text processing now treats complex scripts as a first
class citizen and supports advanced typography at no additional cost.
BUG= 561099
Committed: https://crrev.com/054a96857c077c00cb8b9c71e0a13a9051caa4f7
Cr-Commit-Position: refs/heads/master@{#427685}
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+35 lines, -2420 lines) |
Patch |
|
M |
third_party/WebKit/Source/core/inspector/InspectorCSSAgent.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
12
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/layout/LayoutText.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/layout/LayoutText.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
12
|
1 chunk |
+1 line, -4 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/layout/api/LineLayoutText.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
2 chunks |
+0 lines, -5 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
12
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/BUILD.gn
|
View
|
1
2
3
4
5
6
7
8
9
10
11
12
|
4 chunks |
+0 lines, -7 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/Font.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
4 chunks |
+0 lines, -22 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/Font.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
7 chunks |
+10 lines, -375 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/GlyphBufferTest.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -8 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/Source/platform/fonts/GlyphPage.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -232 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/Source/platform/fonts/GlyphPageTreeNode.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -219 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/Source/platform/fonts/GlyphPageTreeNode.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -485 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/Source/platform/fonts/GlyphPageTreeNodeTest.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
12
|
1 chunk |
+0 lines, -290 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/SegmentedFontData.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
2 chunks |
+3 lines, -4 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/SegmentedFontData.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -4 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/SimpleFontData.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
5 chunks |
+11 lines, -14 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/SimpleFontData.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
8 chunks |
+6 lines, -59 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/UTF16TextIterator.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
2 chunks |
+2 lines, -6 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/UTF16TextIterator.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+1 line, -11 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/opentype/OpenTypeVerticalData.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzShaperTest.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/fonts/shaping/Shaper.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
D |
third_party/WebKit/Source/platform/fonts/shaping/SimpleShaper.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -81 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/Source/platform/fonts/shaping/SimpleShaper.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -254 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/text/Character.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
2 chunks |
+0 lines, -6 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/text/Character.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -136 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/text/CharacterTest.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
2 chunks |
+0 lines, -143 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/Source/platform/text/TextPath.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+0 lines, -39 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/text/TextRun.h
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/web/FrameLoaderClientImpl.cpp
|
View
|
1
2
3
4
5
6
7
8
9
10
11
|
2 chunks |
+0 lines, -8 lines |
0 comments
|
Download
|
Total messages: 96 (46 generated)
|