Delete dead code in deprecated flexbox for RTL line-clamp
Created attachment 158964 [details] Patch
Comment on attachment 158964 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=158964&action=review > Source/WebCore/ChangeLog:9 > + -webkit-line-clamp and direction:rtl is not supported. Delete some > + obviously dead code that half-tries to support it. Does this change cause a visible difference? It's probably OK if it does, but we should still have a test case to show it.
Comment on attachment 158964 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=158964&action=review >> Source/WebCore/ChangeLog:9 >> + obviously dead code that half-tries to support it. > > Does this change cause a visible difference? It's probably OK if it does, but we should still have a test case to show it. No. It's actually just deleting dead code. If you look like 3 lines above you can see that leftToRight is always true.
Comment on attachment 158964 [details] Patch Clearing flags on attachment: 158964 Committed r125929: <http://trac.webkit.org/changeset/125929>
All reviewed patches have been landed. Closing bug.