diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-01-23 13:54:53 -0500 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-01-23 19:48:54 +0000 |
| commit | 6414b273506f97012f82c8a32c020b483d5df31a (patch) | |
| tree | f790a25f03e78fe084ee22cb16418d95deaf3dd3 /xfa/fde/css/cfde_cssstyleselector.cpp | |
| parent | a0b2d23d1121202d3821291483943a47a3c9e32e (diff) | |
| download | pdfium-6414b273506f97012f82c8a32c020b483d5df31a.tar.xz | |
Cleanup CSS parsing
This Cl removes unused CSS parsing code. The only caller of
CFDE_CSSStyleSheet::LoadeBuffer is from CXFA_TextParser which provides a static
string. So, we know there are no Media rules, Import rules, FontFace rules or
Page rules in the input style. These rules are also not allowed in inline
styles. This CL removes the supporting code.
Change-Id: I3a559352b2dd0769c465cbd3da40d1db5f9577b0
Reviewed-on: https://pdfium-review.googlesource.com/2330
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fde/css/cfde_cssstyleselector.cpp')
| -rw-r--r-- | xfa/fde/css/cfde_cssstyleselector.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fde/css/cfde_cssstyleselector.cpp b/xfa/fde/css/cfde_cssstyleselector.cpp index 1539894017..8452c78ff5 100644 --- a/xfa/fde/css/cfde_cssstyleselector.cpp +++ b/xfa/fde/css/cfde_cssstyleselector.cpp @@ -100,13 +100,13 @@ void CFDE_CSSStyleSelector::SetStylePriority( m_ePriorities[static_cast<int32_t>(ePriority)] = eType; } -void CFDE_CSSStyleSelector::UpdateStyleIndex(uint32_t dwMediaList) { +void CFDE_CSSStyleSelector::UpdateStyleIndex() { Reset(); // TODO(dsinclair): Hard coded size bad. This should probably just be a map. for (int32_t iGroup = 0; iGroup < 3; ++iGroup) { CFDE_CSSRuleCollection& rules = m_RuleCollection[iGroup]; - rules.AddRulesFrom(m_SheetGroups[iGroup], dwMediaList, m_pFontMgr); + rules.AddRulesFrom(m_SheetGroups[iGroup], m_pFontMgr); } } |