diff options
author | Ryan Harrison <rharrison@chromium.org> | 2018-02-01 17:46:34 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-01 17:46:34 +0000 |
commit | 55c62a48d36c042499c68b0aca60bc34fa2b5c99 (patch) | |
tree | 0709123d6f7b08a55de4099fa726a805805f08f5 /core/fxcrt/css/cfx_cssstylesheet.cpp | |
parent | 71a7d377ff36a0be1af1848d5cac0ccb83ae725d (diff) | |
download | pdfium-55c62a48d36c042499c68b0aca60bc34fa2b5c99.tar.xz |
Convert CFX_CSSPropertyTable to CFX_CSSData
This converts the existing class naming and references. The ::Entry
struct is converted to ::Property. The GetBy* methods are renamed to
GetPropertyBy*.
Free functions from other parts of the CSS code base are merged into
the class. GetCSSPropertyValueByName becomes ::GetPropertyValueByName
with a return value of ::PropertyValue*. GetCSSLengthUnitByName
becomes ::GetLengthUnitByName with a return value of
::LengthUnit*. GetCSSColorByName becomes ::GetColorByName with a
return value of ::Color*.
BUG=pdfium:998
Change-Id: I5b78c69d0fe53ad714ac3545cfdda8c26b95e4d5
Reviewed-on: https://pdfium-review.googlesource.com/25010
Reviewed-by: dsinclair <dsinclair@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'core/fxcrt/css/cfx_cssstylesheet.cpp')
-rw-r--r-- | core/fxcrt/css/cfx_cssstylesheet.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/core/fxcrt/css/cfx_cssstylesheet.cpp b/core/fxcrt/css/cfx_cssstylesheet.cpp index 01c68fce7d..c7300fb37c 100644 --- a/core/fxcrt/css/cfx_cssstylesheet.cpp +++ b/core/fxcrt/css/cfx_cssstylesheet.cpp @@ -8,8 +8,8 @@ #include <utility> +#include "core/fxcrt/css/cfx_cssdata.h" #include "core/fxcrt/css/cfx_cssdeclaration.h" -#include "core/fxcrt/css/cfx_csspropertytable.h" #include "core/fxcrt/css/cfx_cssstylerule.h" #include "core/fxcrt/fx_codepage.h" #include "third_party/base/ptr_util.h" @@ -62,7 +62,7 @@ CFX_CSSSyntaxStatus CFX_CSSStyleSheet::LoadStyleRule( CFX_CSSStyleRule* pStyleRule = nullptr; int32_t iValueLen = 0; - const CFX_CSSPropertyTable::Entry* entry = nullptr; + const CFX_CSSData::Property* property = nullptr; WideString wsName; while (1) { switch (pSyntax->DoSyntaxParse()) { @@ -75,18 +75,18 @@ CFX_CSSSyntaxStatus CFX_CSSStyleSheet::LoadStyleRule( } case CFX_CSSSyntaxStatus::PropertyName: { WideStringView strValue = pSyntax->GetCurrentString(); - entry = CFX_CSSPropertyTable::GetByName(strValue); - if (!entry) + property = CFX_CSSData::GetPropertyByName(strValue); + if (!property) wsName = WideString(strValue); break; } case CFX_CSSSyntaxStatus::PropertyValue: { - if (entry || iValueLen > 0) { + if (property || iValueLen > 0) { WideStringView strValue = pSyntax->GetCurrentString(); auto* decl = pStyleRule->GetDeclaration(); if (!strValue.IsEmpty()) { - if (entry) { - decl->AddProperty(entry, strValue); + if (property) { + decl->AddProperty(property, strValue); } else { decl->AddProperty(wsName, WideString(strValue)); } |