From 1e2ece0cf98f8daecdea5b45ebd20cedbcef4bd9 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Tue, 11 Aug 2015 13:54:33 -0700 Subject: Remove dead code from CPDF_Metadata. Add missing nullptr check. BUG=pdfium:117 R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1282653002 . --- core/src/fpdfdoc/doc_metadata.cpp | 137 ++++---------------------------------- 1 file changed, 13 insertions(+), 124 deletions(-) (limited to 'core/src') diff --git a/core/src/fpdfdoc/doc_metadata.cpp b/core/src/fpdfdoc/doc_metadata.cpp index b617eecf87..c779f68354 100644 --- a/core/src/fpdfdoc/doc_metadata.cpp +++ b/core/src/fpdfdoc/doc_metadata.cpp @@ -6,134 +6,23 @@ #include "../../include/fpdfdoc/fpdf_doc.h" #include "../../include/fxcrt/fx_xml.h" -typedef struct _PDFDOC_METADATA { - CPDF_Document* m_pDoc; - CXML_Element* m_pXmlElmnt; - CXML_Element* m_pElmntRdf; - CFX_CMapByteStringToPtr* m_pStringMap; -} PDFDOC_METADATA, *PDFDOC_LPMETADATA; -typedef PDFDOC_METADATA const* PDFDOC_LPCMETADATA; -const FX_CHAR* const gs_FPDFDOC_Metadata_Titles[] = { - "Title", "title", "Subject", "description", "Author", - "creator", "Keywords", "Keywords", "Producer", "Producer", - "Creator", "CreatorTool", "CreationDate", "CreateDate", "ModDate", - "ModifyDate", "MetadataDate", "MetadataDate"}; -CPDF_Metadata::CPDF_Metadata() { - m_pData = FX_Alloc(PDFDOC_METADATA, 1); - CFX_CMapByteStringToPtr*& pStringMap = - ((PDFDOC_LPMETADATA)m_pData)->m_pStringMap; - pStringMap = new CFX_CMapByteStringToPtr; - CFX_ByteString bstr; - for (int i = 0; i < 18; i += 2) { - bstr = gs_FPDFDOC_Metadata_Titles[i]; - pStringMap->AddValue(bstr, (void*)gs_FPDFDOC_Metadata_Titles[i + 1]); - } -} -CPDF_Metadata::~CPDF_Metadata() { - FXSYS_assert(m_pData != NULL); - CXML_Element*& p = ((PDFDOC_LPMETADATA)m_pData)->m_pXmlElmnt; - delete p; - CFX_CMapByteStringToPtr* pStringMap = - ((PDFDOC_LPMETADATA)m_pData)->m_pStringMap; - if (pStringMap) { - pStringMap->RemoveAll(); - delete pStringMap; - } - FX_Free(m_pData); -} -void CPDF_Metadata::LoadDoc(CPDF_Document* pDoc) { - FXSYS_assert(pDoc != NULL); - ((PDFDOC_LPMETADATA)m_pData)->m_pDoc = pDoc; + +CPDF_Metadata::CPDF_Metadata(CPDF_Document* pDoc) : m_pDoc(pDoc) { CPDF_Dictionary* pRoot = pDoc->GetRoot(); + if (!pRoot) + return; + CPDF_Stream* pStream = pRoot->GetStream(FX_BSTRC("Metadata")); - if (!pStream) { + if (!pStream) return; - } + CPDF_StreamAcc acc; acc.LoadAllData(pStream, FALSE); - int size = acc.GetSize(); - const uint8_t* pBuf = acc.GetData(); - CXML_Element*& pXmlElmnt = ((PDFDOC_LPMETADATA)m_pData)->m_pXmlElmnt; - pXmlElmnt = CXML_Element::Parse(pBuf, size); - if (!pXmlElmnt) { - return; - } - CXML_Element*& pElmntRdf = ((PDFDOC_LPMETADATA)m_pData)->m_pElmntRdf; - if (pXmlElmnt->GetTagName() == FX_BSTRC("RDF")) { - pElmntRdf = pXmlElmnt; - } else { - pElmntRdf = pXmlElmnt->GetElement(NULL, FX_BSTRC("RDF")); - } -} -int32_t CPDF_Metadata::GetString(const CFX_ByteStringC& bsItem, - CFX_WideString& wsStr) { - if (!((PDFDOC_LPMETADATA)m_pData)->m_pXmlElmnt) { - return -1; - } - if (!((PDFDOC_LPMETADATA)m_pData)->m_pStringMap) { - return -1; - } - void* szTag; - if (!((PDFDOC_LPMETADATA)m_pData)->m_pStringMap->Lookup(bsItem, szTag)) { - return -1; - } - CFX_ByteString bsTag = (const FX_CHAR*)szTag; - wsStr = L""; - CXML_Element* pElmntRdf = ((PDFDOC_LPMETADATA)m_pData)->m_pElmntRdf; - if (!pElmntRdf) { - return -1; - } - int nChild = pElmntRdf->CountChildren(); - for (int i = 0; i < nChild; i++) { - CXML_Element* pTag = - pElmntRdf->GetElement(NULL, FX_BSTRC("Description"), i); - if (!pTag) { - continue; - } - if (bsItem == FX_BSTRC("Title") || bsItem == FX_BSTRC("Subject")) { - CXML_Element* pElmnt = pTag->GetElement(NULL, bsTag); - if (!pElmnt) { - continue; - } - pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("Alt")); - if (!pElmnt) { - continue; - } - pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("li")); - if (!pElmnt) { - continue; - } - wsStr = pElmnt->GetContent(0); - return wsStr.GetLength(); - } - if (bsItem == FX_BSTRC("Author")) { - CXML_Element* pElmnt = pTag->GetElement(NULL, bsTag); - if (!pElmnt) { - continue; - } - pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("Seq")); - if (!pElmnt) { - continue; - } - pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("li")); - if (!pElmnt) { - continue; - } - wsStr = pElmnt->GetContent(0); - return wsStr.GetLength(); - } - CXML_Element* pElmnt = pTag->GetElement(NULL, bsTag); - if (!pElmnt) { - continue; - } - wsStr = pElmnt->GetContent(0); - return wsStr.GetLength(); - } - return -1; -} -CXML_Element* CPDF_Metadata::GetRoot() const { - return ((PDFDOC_LPMETADATA)m_pData)->m_pXmlElmnt; + m_pXmlElement.reset(CXML_Element::Parse(acc.GetData(), acc.GetSize())); } -CXML_Element* CPDF_Metadata::GetRDF() const { - return ((PDFDOC_LPMETADATA)m_pData)->m_pElmntRdf; + +CPDF_Metadata::~CPDF_Metadata() {} + +const CXML_Element* CPDF_Metadata::GetRoot() const { + return m_pXmlElement.get(); } -- cgit v1.2.3