diff options
author | Lei Zhang <thestig@chromium.org> | 2018-10-18 20:51:54 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-10-18 20:51:54 +0000 |
commit | 8e49f3fce1a56634284b254a0dc2d329da349355 (patch) | |
tree | 153842a78192a5726376efadeff7c2be69ec4191 /core | |
parent | 4754adaf70d8708610ea31a4d5e3df7c4fcb2ef0 (diff) | |
download | pdfium-8e49f3fce1a56634284b254a0dc2d329da349355.tar.xz |
Do IWYU in core/fxcrt/css and fix lint errors.
Change-Id: I6d2b35849e9f1935277986b6c72e6f507d976974
Reviewed-on: https://pdfium-review.googlesource.com/c/43948
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'core')
-rw-r--r-- | core/fxcrt/css/cfx_css.h | 2 | ||||
-rw-r--r-- | core/fxcrt/css/cfx_csscolorvalue.h | 1 | ||||
-rw-r--r-- | core/fxcrt/css/cfx_csscomputedstyle.cpp | 10 | ||||
-rw-r--r-- | core/fxcrt/css/cfx_csscomputedstyle.h | 3 | ||||
-rw-r--r-- | core/fxcrt/css/cfx_cssdata.h | 1 | ||||
-rw-r--r-- | core/fxcrt/css/cfx_cssstringvalue.h | 1 | ||||
-rw-r--r-- | core/fxcrt/css/cfx_cssvalue.h | 1 | ||||
-rw-r--r-- | core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp | 1 |
8 files changed, 13 insertions, 7 deletions
diff --git a/core/fxcrt/css/cfx_css.h b/core/fxcrt/css/cfx_css.h index 5b70d0b2e1..30d9ff80fb 100644 --- a/core/fxcrt/css/cfx_css.h +++ b/core/fxcrt/css/cfx_css.h @@ -7,7 +7,7 @@ #ifndef CORE_FXCRT_CSS_CFX_CSS_H_ #define CORE_FXCRT_CSS_CFX_CSS_H_ -#include "core/fxge/fx_dib.h" +#include <stdint.h> enum CFX_CSSVALUETYPE { CFX_CSSVALUETYPE_Primitive = 1 << 0, diff --git a/core/fxcrt/css/cfx_csscolorvalue.h b/core/fxcrt/css/cfx_csscolorvalue.h index 0156caa786..fdc3fcbe2f 100644 --- a/core/fxcrt/css/cfx_csscolorvalue.h +++ b/core/fxcrt/css/cfx_csscolorvalue.h @@ -8,6 +8,7 @@ #define CORE_FXCRT_CSS_CFX_CSSCOLORVALUE_H_ #include "core/fxcrt/css/cfx_cssvalue.h" +#include "core/fxge/fx_dib.h" class CFX_CSSColorValue final : public CFX_CSSValue { public: diff --git a/core/fxcrt/css/cfx_csscomputedstyle.cpp b/core/fxcrt/css/cfx_csscomputedstyle.cpp index eff708122b..11ae901185 100644 --- a/core/fxcrt/css/cfx_csscomputedstyle.cpp +++ b/core/fxcrt/css/cfx_csscomputedstyle.cpp @@ -9,16 +9,16 @@ #include "core/fxcrt/css/cfx_cssstringvalue.h" #include "core/fxcrt/css/cfx_cssvaluelist.h" -CFX_CSSComputedStyle::CFX_CSSComputedStyle() {} +CFX_CSSComputedStyle::CFX_CSSComputedStyle() = default; -CFX_CSSComputedStyle::~CFX_CSSComputedStyle() {} +CFX_CSSComputedStyle::~CFX_CSSComputedStyle() = default; bool CFX_CSSComputedStyle::GetCustomStyle(const WideString& wsName, - WideString& wsValue) const { + WideString* pValue) const { for (auto iter = m_CustomProperties.rbegin(); - iter != m_CustomProperties.rend(); iter++) { + iter != m_CustomProperties.rend(); ++iter) { if (wsName == iter->name()) { - wsValue = iter->value(); + *pValue = iter->value(); return true; } } diff --git a/core/fxcrt/css/cfx_csscomputedstyle.h b/core/fxcrt/css/cfx_csscomputedstyle.h index f97f3f1d7d..d4959eef06 100644 --- a/core/fxcrt/css/cfx_csscomputedstyle.h +++ b/core/fxcrt/css/cfx_csscomputedstyle.h @@ -12,6 +12,7 @@ #include "core/fxcrt/css/cfx_css.h" #include "core/fxcrt/css/cfx_csscustomproperty.h" #include "core/fxcrt/fx_string.h" +#include "core/fxge/fx_dib.h" class CFX_CSSValueList; @@ -94,7 +95,7 @@ class CFX_CSSComputedStyle final : public Retainable { void SetLetterSpacing(const CFX_CSSLength& letterSpacing); void AddCustomStyle(const CFX_CSSCustomProperty& prop); - bool GetCustomStyle(const WideString& wsName, WideString& wsValue) const; + bool GetCustomStyle(const WideString& wsName, WideString* pValue) const; InheritedData m_InheritedData; NonInheritedData m_NonInheritedData; diff --git a/core/fxcrt/css/cfx_cssdata.h b/core/fxcrt/css/cfx_cssdata.h index a2f774b1af..8e1d265edf 100644 --- a/core/fxcrt/css/cfx_cssdata.h +++ b/core/fxcrt/css/cfx_cssdata.h @@ -11,6 +11,7 @@ #include "core/fxcrt/css/cfx_cssnumbervalue.h" #include "core/fxcrt/css/cfx_cssvalue.h" #include "core/fxcrt/string_view_template.h" +#include "core/fxge/fx_dib.h" class CFX_CSSData { public: diff --git a/core/fxcrt/css/cfx_cssstringvalue.h b/core/fxcrt/css/cfx_cssstringvalue.h index d49393c305..b59cef097e 100644 --- a/core/fxcrt/css/cfx_cssstringvalue.h +++ b/core/fxcrt/css/cfx_cssstringvalue.h @@ -8,6 +8,7 @@ #define CORE_FXCRT_CSS_CFX_CSSSTRINGVALUE_H_ #include "core/fxcrt/css/cfx_cssvalue.h" +#include "core/fxcrt/widestring.h" class CFX_CSSStringValue final : public CFX_CSSValue { public: diff --git a/core/fxcrt/css/cfx_cssvalue.h b/core/fxcrt/css/cfx_cssvalue.h index f143e96036..30aace2b64 100644 --- a/core/fxcrt/css/cfx_cssvalue.h +++ b/core/fxcrt/css/cfx_cssvalue.h @@ -8,6 +8,7 @@ #define CORE_FXCRT_CSS_CFX_CSSVALUE_H_ #include "core/fxcrt/css/cfx_css.h" +#include "core/fxcrt/retain_ptr.h" class CFX_CSSValue : public Retainable { public: diff --git a/core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp b/core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp index e232a2dc65..9d19d275f6 100644 --- a/core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp +++ b/core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp @@ -6,6 +6,7 @@ #include "core/fxcrt/css/cfx_cssvaluelistparser.h" +#include "core/fxcrt/widestring.h" #include "testing/gtest/include/gtest/gtest.h" #include "testing/test_support.h" #include "third_party/base/ptr_util.h" |