summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2018-10-18 20:51:54 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-10-18 20:51:54 +0000
commit8e49f3fce1a56634284b254a0dc2d329da349355 (patch)
tree153842a78192a5726376efadeff7c2be69ec4191 /core
parent4754adaf70d8708610ea31a4d5e3df7c4fcb2ef0 (diff)
downloadpdfium-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.h2
-rw-r--r--core/fxcrt/css/cfx_csscolorvalue.h1
-rw-r--r--core/fxcrt/css/cfx_csscomputedstyle.cpp10
-rw-r--r--core/fxcrt/css/cfx_csscomputedstyle.h3
-rw-r--r--core/fxcrt/css/cfx_cssdata.h1
-rw-r--r--core/fxcrt/css/cfx_cssstringvalue.h1
-rw-r--r--core/fxcrt/css/cfx_cssvalue.h1
-rw-r--r--core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp1
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"