diff options
author | Ryan Harrison <rharrison@chromium.org> | 2018-01-31 15:47:32 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-31 15:47:32 +0000 |
commit | 41b45a88f14ea546d1b46b7c30a1c9c1db778ac3 (patch) | |
tree | ffad96b6e33807ed3f78056c9ba879e1c3dceab3 /core/fxcrt/css/cfx_cssstylesheet.cpp | |
parent | 3f3f5130e91c9a868831ee80840f027111ee6f3b (diff) | |
download | pdfium-41b45a88f14ea546d1b46b7c30a1c9c1db778ac3.tar.xz |
Clean up CSS property table
Reorganizes the free functions to be static methods on a
class. Converts the Table struct to be an Entry struct defined in the
class.
Change-Id: I9eca0dfb15187af10cbe2dc4b4c2756d55d957be
Reviewed-on: https://pdfium-review.googlesource.com/24712
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: dsinclair <dsinclair@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 183765f3b0..01c68fce7d 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_cssdatatable.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* propertyTable = nullptr; + const CFX_CSSPropertyTable::Entry* entry = 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(); - propertyTable = CFX_GetCSSPropertyByName(strValue); - if (!propertyTable) + entry = CFX_CSSPropertyTable::GetByName(strValue); + if (!entry) wsName = WideString(strValue); break; } case CFX_CSSSyntaxStatus::PropertyValue: { - if (propertyTable || iValueLen > 0) { + if (entry || iValueLen > 0) { WideStringView strValue = pSyntax->GetCurrentString(); auto* decl = pStyleRule->GetDeclaration(); if (!strValue.IsEmpty()) { - if (propertyTable) { - decl->AddProperty(propertyTable, strValue); + if (entry) { + decl->AddProperty(entry, strValue); } else { decl->AddProperty(wsName, WideString(strValue)); } |