Improve the performance of determining character boundaries in ComplexTextController
Created attachment 341258 [details] Patch
Created attachment 341259 [details] Patch
16x faster, not 4x faster.
Comment on attachment 341259 [details] Patch Attachment 341259 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/7797104 New failing tests: fast/text/arabic-zwj-and-zwnj.html
Created attachment 341261 [details] Archive of layout-test-results from ews103 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-sierra Platform: Mac OS X 10.12.6
Comment on attachment 341259 [details] Patch Attachment 341259 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/7797112 New failing tests: fast/text/arabic-zwj-and-zwnj.html
Created attachment 341263 [details] Archive of layout-test-results from ews106 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
Comment on attachment 341259 [details] Patch Attachment 341259 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/7797137 New failing tests: fast/text/arabic-zwj-and-zwnj.html
Created attachment 341265 [details] Archive of layout-test-results from ews117 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews117 Port: mac-sierra Platform: Mac OS X 10.12.6
The test is failing because createAndFillGlyphPage() maps zeroWidthNonJoiner (which GeezaPro supports) to zeroWidthSpace (which GeezaPro doesn't support)
Created attachment 341370 [details] WIP
Attachment 341370 [details] did not pass style-queue: ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 1 in 9 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 341370 [details] WIP Attachment 341370 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/7807537 New failing tests: fast/text/arabic-zwj-and-zwnj.html fast/text/format-control.html
Created attachment 341375 [details] Archive of layout-test-results from ews101 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews101 Port: mac-sierra Platform: Mac OS X 10.12.6
Comment on attachment 341370 [details] WIP Attachment 341370 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/7807676 New failing tests: imported/blink/fast/text/international/zerowidthjoiner-should-not-render.html
Created attachment 341379 [details] Archive of layout-test-results from ews125 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews125 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.4
Comment on attachment 341370 [details] WIP Attachment 341370 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/7807721 New failing tests: fast/text/arabic-zwj-and-zwnj.html fast/text/format-control.html
Created attachment 341380 [details] Archive of layout-test-results from ews114 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews114 Port: mac-sierra Platform: Mac OS X 10.12.6
Comment on attachment 341370 [details] WIP Attachment 341370 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/7809084 New failing tests: fast/text/arabic-zwj-and-zwnj.html fast/text/format-control.html
Created attachment 341387 [details] Archive of layout-test-results from ews104 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
Created attachment 341622 [details] WIP
Attachment 341622 [details] did not pass style-queue: ERROR: Source/WebCore/platform/graphics/Font.cpp:204: Should have a space between // and comment [whitespace/comments] [4] ERROR: Source/WebCore/platform/graphics/Font.cpp:205: Should have a space between // and comment [whitespace/comments] [4] Total errors found: 2 in 11 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 341622 [details] WIP Attachment 341622 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/7881306 New failing tests: fast/text/arabic-zwj-and-zwnj.html fast/text/format-control.html
Created attachment 341630 [details] Archive of layout-test-results from ews100 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-sierra Platform: Mac OS X 10.12.6
Comment on attachment 341622 [details] WIP Attachment 341622 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/7881004 New failing tests: imported/blink/fast/text/international/zerowidthjoiner-should-not-render.html
Created attachment 341632 [details] Archive of layout-test-results from ews121 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews121 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.4
Comment on attachment 341622 [details] WIP Attachment 341622 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/7881340 New failing tests: fast/text/arabic-zwj-and-zwnj.html fast/text/format-control.html
Created attachment 341634 [details] Archive of layout-test-results from ews115 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews115 Port: mac-sierra Platform: Mac OS X 10.12.6
Comment on attachment 341622 [details] WIP Attachment 341622 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/7881757 New failing tests: fast/text/arabic-zwj-and-zwnj.html fast/text/format-control.html
Created attachment 341635 [details] Archive of layout-test-results from ews106 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
Created attachment 341667 [details] WIP
Attachment 341667 [details] did not pass style-queue: ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 1 in 14 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 341667 [details] WIP Attachment 341667 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/7901053 New failing tests: fast/text/soft-hyphen-min-preferred-width.html
Created attachment 341670 [details] Archive of layout-test-results from ews101 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews101 Port: mac-sierra Platform: Mac OS X 10.12.6
Comment on attachment 341667 [details] WIP Attachment 341667 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/7901054 New failing tests: fast/text/soft-hyphen-min-preferred-width.html
Created attachment 341671 [details] Archive of layout-test-results from ews104 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
Comment on attachment 341667 [details] WIP Attachment 341667 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/7901547 New failing tests: fast/text/soft-hyphen-min-preferred-width.html
Created attachment 341675 [details] Archive of layout-test-results from ews115 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews115 Port: mac-sierra Platform: Mac OS X 10.12.6
Created attachment 341718 [details] Patch
<rdar://problem/37912480>
Comment on attachment 341718 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=341718&action=review > LayoutTests/ChangeLog:14 > + * platform/mac/fast/text/format-control-expected.png: > + * platform/mac/fast/text/format-control-expected.txt: Rebaseline. This test is regressing! For example, the presence of a zero-width non-joiner is no longer preventing the fi ligature from forming on the second line.
Comment on attachment 341718 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=341718&action=review > Source/WebCore/platform/graphics/Font.cpp:-203 > - overwriteCodePoint(leftToRightMark, zeroWidthSpace); > - overwriteCodePoint(rightToLeftMark, zeroWidthSpace); > - overwriteCodePoint(leftToRightEmbed, zeroWidthSpace); > - overwriteCodePoint(rightToLeftEmbed, zeroWidthSpace); > - overwriteCodePoint(leftToRightOverride, zeroWidthSpace); > - overwriteCodePoint(rightToLeftOverride, zeroWidthSpace); > - overwriteCodePoint(leftToRightIsolate, zeroWidthSpace); > - overwriteCodePoint(rightToLeftIsolate, zeroWidthSpace); I think this may regress behavior with fonts that have glyphs for these control characters (I think Arial Unicode from Microsoft is an example of that).
*** This bug has been marked as a duplicate of bug 175778 ***