Cache glyph paths and share underline skipping code between all the ports
Created attachment 353814 [details] Patch
Created attachment 353816 [details] Patch
Created attachment 353819 [details] Patch
Created attachment 353821 [details] Patch
Created attachment 353830 [details] Patch
Created attachment 353881 [details] Patch
Comment on attachment 353881 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=353881&action=review > Source/WebCore/platform/graphics/FontCascade.cpp:1241 > +enum class GlyphUnderlineType { : uint8_t > Source/WebCore/platform/graphics/FontCascade.cpp:1669 > + GlyphIterationState(FloatPoint startingPoint, FloatPoint currentPoint, float y1, float y2, float minX, float maxX) It seems like this constructor could be omitted by using an initializer list where this constructor is called.
Created attachment 353916 [details] Patch for committing
Comment on attachment 353881 [details] Patch Attachment 353881 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: https://webkit-queues.webkit.org/results/9870516 New failing tests: imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-setRemoteDescription-tracks.https.html
Created attachment 353920 [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.6
Created attachment 353921 [details] Patch for committing
Committed r237845: <https://trac.webkit.org/changeset/237845>
<rdar://problem/45831149>
It broke the underscore beneath hyperlinks, bug #191630.
Comment on attachment 353881 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=353881&action=review > Source/WebCore/platform/graphics/FontCascade.cpp:1751 > + , m_translation(AffineTransform::translation(textOrigin.x(), textOrigin.y()).scale(1, -1)) This is what broke cairo ports, we didn't have that scale(1, -1) in the cairo impl.