DescriptionRevert of Add OpenTypeCapsSupport class (patchset #3 id:40001 of https://codereview.chromium.org/1882063002/ )
Reason for revert:
breaks layout tests on Linux
6 tests crashed:
OpenTypeCapsSupportTest.LibertineAllPetiteSynthesis (../../third_party/WebKit/Source/platform/fonts/opentype/OpenTypeCapsSupportTest.cpp:53)
OpenTypeCapsSupportTest.LibertineIgnoreMissingTitling (../../third_party/WebKit/Source/platform/fonts/opentype/OpenTypeCapsSupportTest.cpp:38)
OpenTypeCapsSupportTest.LibertineSmcpC2scSupported (../../third_party/WebKit/Source/platform/fonts/opentype/OpenTypeCapsSupportTest.cpp:23)
OpenTypeCapsSupportTest.LibertineUnicaseFallback (../../third_party/WebKit/Source/platform/fonts/opentype/OpenTypeCapsSupportTest.cpp:98)
OpenTypeCapsSupportTest.MEgalopolisSmallCapsSynthetic (../../third_party/WebKit/Source/platform/fonts/opentype/OpenTypeCapsSupportTest.cpp:68)
OpenTypeCapsSupportTest.MEgalopolisUnicaseSynthetic (../../third_party/WebKit/Source/platform/fonts/opentype/OpenTypeCapsSupportTest.cpp:83)
Original issue's description:
> Add OpenTypeCapsSupport class
>
> As preparation for implementing correct font-variant-caps we need to be
> able to check font files for their ability to support caps related
> OpenType features, such as 'smcp', 'c2sc' etc.
>
> The implementation files are split in two, one part Chromium-side
> implementation, one part Mozilla Public License'd code adapted from
> Firefox.
>
> BUG=587094
> TEST=OpenTypeCapsSupportTest.cpp
> TBR=dpranke
>
> Committed: https://crrev.com/530c73184ff5fed64d9dae5ff1d452d1a75d3a99
> Cr-Commit-Position: refs/heads/master@{#387085}
TBR=behdad@chromium.org,dpranke@chromium.org,eae@chromium.org,kojii@chromium.org,drott@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=587094
Committed: https://crrev.com/3725b50a27ae6ba927cd0cd0e6d79eae86d9bf73
Cr-Commit-Position: refs/heads/master@{#387119}
Patch Set 1 #
Messages
Total messages: 7 (2 generated)
|