diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2017-01-23 15:37:39 -0500 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-01-24 13:59:50 +0000 |
commit | e73fea598f088151213fb11100798c615543ca2f (patch) | |
tree | 6d44f161ad24c7705443be0eed802fac5151f9f9 /xfa/fde/css/cfde_cssstylesheet_unittest.cpp | |
parent | 35a89dde15007d28c6796926708e8f0b5ee60d74 (diff) | |
download | pdfium-e73fea598f088151213fb11100798c615543ca2f.tar.xz |
Remove CSS Pseudo parsing
We always match against the pseudo type NONE when matching selectors, so we
never end up using any pseudo selectors. This CL removes the pseudo selector
parsing code.
Change-Id: I7831d12dfff3a6f1dc98ff8e1d63c1090775562c
Reviewed-on: https://pdfium-review.googlesource.com/2294
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fde/css/cfde_cssstylesheet_unittest.cpp')
-rw-r--r-- | xfa/fde/css/cfde_cssstylesheet_unittest.cpp | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/xfa/fde/css/cfde_cssstylesheet_unittest.cpp b/xfa/fde/css/cfde_cssstylesheet_unittest.cpp index 1c50f65607..048eeb7c84 100644 --- a/xfa/fde/css/cfde_cssstylesheet_unittest.cpp +++ b/xfa/fde/css/cfde_cssstylesheet_unittest.cpp @@ -139,18 +139,6 @@ TEST_F(CFDE_CSSStyleSheetTest, ParseMultipleSelectorsCombined) { LoadAndVerifyDecl(L"a, b, c { border: 5px; }", {L"a", L"b", L"c"}, 4); } -TEST_F(CFDE_CSSStyleSheetTest, ParseWithPseudo) { - // TODO(dsinclair): I think this is wrong, as the selector just becomes - // :before and we lose the a? - LoadAndVerifyDecl(L"a:before { border: 10px; }", {L":before"}, 4); -} - -TEST_F(CFDE_CSSStyleSheetTest, ParseWithSelectorsAndPseudo) { - // TODO(dsinclair): I think this is wrong as we lose the b on the b:before - LoadAndVerifyDecl(L"a, b:before, c { border: 1px; }", - {L"a", L":before", L"c"}, 4); -} - TEST_F(CFDE_CSSStyleSheetTest, ParseBorder) { LoadAndVerifyDecl(L"a { border: 5px; }", {L"a"}, 4); VerifyFloat(FDE_CSSProperty::BorderLeftWidth, 5.0, FDE_CSSNumberType::Pixels); |