From 8e49f3fce1a56634284b254a0dc2d329da349355 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Thu, 18 Oct 2018 20:51:54 +0000 Subject: 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 Reviewed-by: Ryan Harrison --- core/fxcrt/css/cfx_css.h | 2 +- core/fxcrt/css/cfx_csscolorvalue.h | 1 + core/fxcrt/css/cfx_csscomputedstyle.cpp | 10 +++++----- core/fxcrt/css/cfx_csscomputedstyle.h | 3 ++- core/fxcrt/css/cfx_cssdata.h | 1 + core/fxcrt/css/cfx_cssstringvalue.h | 1 + core/fxcrt/css/cfx_cssvalue.h | 1 + core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp | 1 + 8 files changed, 13 insertions(+), 7 deletions(-) (limited to 'core') 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 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" -- cgit v1.2.3