From 86adb658843658969041a13bf6cf0cc950ffe8ed Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Thu, 15 Oct 2015 14:51:42 -0700 Subject: Fix layering violation in CPDF_Document::FromFPDFDocument(). Make this a function in the fpdfsdk/ layer, rather than a method in the core/ layer. Nothing in core should know about public FPDF types. BUG=pdfium:217 R=thestig@chromium.org Review URL: https://codereview.chromium.org/1396283006 . --- core/include/fpdfapi/fpdf_parser.h | 3 --- core/src/fpdfapi/fpdf_parser/fpdf_parser_document.cpp | 4 ---- 2 files changed, 7 deletions(-) (limited to 'core') diff --git a/core/include/fpdfapi/fpdf_parser.h b/core/include/fpdfapi/fpdf_parser.h index b58cf14974..04c7d90f6f 100644 --- a/core/include/fpdfapi/fpdf_parser.h +++ b/core/include/fpdfapi/fpdf_parser.h @@ -7,7 +7,6 @@ #ifndef CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_ #define CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_ -#include "../../../public/fpdfview.h" #include "../fxcrt/fx_system.h" #include "fpdf_objects.h" @@ -57,8 +56,6 @@ extern const FX_WORD PDFDocEncoding[256]; class CPDF_Document : public CFX_PrivateData, public CPDF_IndirectObjects { public: - static CPDF_Document* FromFPDFDocument(FPDF_DOCUMENT document); - CPDF_Document(); explicit CPDF_Document(CPDF_Parser* pParser); diff --git a/core/src/fpdfapi/fpdf_parser/fpdf_parser_document.cpp b/core/src/fpdfapi/fpdf_parser/fpdf_parser_document.cpp index b93cadd266..2dff766187 100644 --- a/core/src/fpdfapi/fpdf_parser/fpdf_parser_document.cpp +++ b/core/src/fpdfapi/fpdf_parser/fpdf_parser_document.cpp @@ -7,10 +7,6 @@ #include "../../../include/fpdfapi/fpdf_parser.h" #include "../../../include/fpdfapi/fpdf_module.h" -// static -CPDF_Document* CPDF_Document::FromFPDFDocument(FPDF_DOCUMENT document) { - return static_cast(document); -} CPDF_Document::CPDF_Document(CPDF_Parser* pParser) : CPDF_IndirectObjects(pParser) { -- cgit v1.2.3