diff options
| author | weili <weili@chromium.org> | 2016-06-18 06:25:37 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-06-18 06:25:37 -0700 |
| commit | eec3a366adbfada36b98f1de651546ee09df8fc0 (patch) | |
| tree | 794aa8c69392b60d8737a8a5b424732b98bdd4b4 /xfa/fde/css/fde_csssyntax.h | |
| parent | 4ce94e118d66064715de5baebeb4b2b580dcac66 (diff) | |
| download | pdfium-eec3a366adbfada36b98f1de651546ee09df8fc0.tar.xz | |
Make code compile with clang_use_chrome_plugin (final)
This change mainly contains files in xfa/ and fxjse/ directories
which were not covered by previous changes.
This is part of the efforts to make PDFium code compilable
by Clang chromium style plugins. After this change, PDFium can be
compiled with "clang_use_chrome_plugin=true" for GN build. Since
clang_use_chrome_plugin is true by default, we no longer need to
set this parameter explicitly.
The changes are mainly the following:
-- move inline constructor/destructor of complex class/struct out-of-line;
-- add constructor/destructor of complex class/struct if not
explicitly defined;
-- add explicit out-of-line copy constructor when needed;
-- move inline virtual functions out-of-line;
-- Properly mark virtual functions with 'override';
-- some minor cleanups;
BUG=pdfium:469
Review-Url: https://codereview.chromium.org/2072803002
Diffstat (limited to 'xfa/fde/css/fde_csssyntax.h')
| -rw-r--r-- | xfa/fde/css/fde_csssyntax.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xfa/fde/css/fde_csssyntax.h b/xfa/fde/css/fde_csssyntax.h index a4e9249055..294dfcb540 100644 --- a/xfa/fde/css/fde_csssyntax.h +++ b/xfa/fde/css/fde_csssyntax.h @@ -14,7 +14,8 @@ class CFDE_CSSTextBuf : public CFX_Target { public: CFDE_CSSTextBuf(); - ~CFDE_CSSTextBuf(); + ~CFDE_CSSTextBuf() override; + FX_BOOL AttachBuffer(const FX_WCHAR* pBuffer, int32_t iBufLen); FX_BOOL EstimateSize(int32_t iAllocSize); int32_t LoadFromStream(IFX_Stream* pTxtStream, |