diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-02-14 15:44:21 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-14 15:44:21 +0000 |
| commit | d8d6f55140bd4f0a67de833346113cfa6e03c4bd (patch) | |
| tree | 1031b18090215b45f2a1533dd0b17390bd6d96cc /xfa/fxfa/parser/cxfa_document.cpp | |
| parent | 2f68cb53f5938e2dd572db79fc5689c987cb350d (diff) | |
| download | pdfium-d8d6f55140bd4f0a67de833346113cfa6e03c4bd.tar.xz | |
Cleanup CXFA_Document const-ness
This CL marks methods const where possible and removes redundant
methods from CXFA_Document.
Change-Id: I34804425469dfd52469daba68f64a4b511e5d4d0
Reviewed-on: https://pdfium-review.googlesource.com/26730
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_document.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_document.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/xfa/fxfa/parser/cxfa_document.cpp b/xfa/fxfa/parser/cxfa_document.cpp index 2a9d80e0cd..01ba3205ed 100644 --- a/xfa/fxfa/parser/cxfa_document.cpp +++ b/xfa/fxfa/parser/cxfa_document.cpp @@ -107,10 +107,6 @@ CXFA_LayoutProcessor* CXFA_Document::GetLayoutProcessor() { return m_pLayoutProcessor.get(); } -CXFA_LayoutProcessor* CXFA_Document::GetDocLayout() { - return GetLayoutProcessor(); -} - void CXFA_Document::ClearLayoutData() { m_pLayoutProcessor.reset(); m_pScriptContext.reset(); @@ -258,7 +254,7 @@ CFXJSE_Engine* CXFA_Document::InitScriptContext(CFXJS_Engine* fxjs_engine) { // We have to call |InitScriptContext| before any calls to |GetScriptContext| // or the context won't have an isolate set into it. -CFXJSE_Engine* CXFA_Document::GetScriptContext() { +CFXJSE_Engine* CXFA_Document::GetScriptContext() const { ASSERT(m_pScriptContext); return m_pScriptContext.get(); } @@ -291,7 +287,7 @@ XFA_VERSION CXFA_Document::RecognizeXFAVersionNumber( } CXFA_Node* CXFA_Document::GetNodeByID(CXFA_Node* pRoot, - const WideStringView& wsID) { + const WideStringView& wsID) const { if (!pRoot || wsID.IsEmpty()) return nullptr; |