summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
Diffstat (limited to 'core')
-rw-r--r--core/fpdfdoc/cpdf_metadata.cpp83
-rw-r--r--core/fpdfdoc/cpdf_metadata_unittest.cpp16
-rw-r--r--core/fxcrt/xml/cfx_xmlparser_unittest.cpp2
3 files changed, 54 insertions, 47 deletions
diff --git a/core/fpdfdoc/cpdf_metadata.cpp b/core/fpdfdoc/cpdf_metadata.cpp
index 11fde82036..972569a25d 100644
--- a/core/fpdfdoc/cpdf_metadata.cpp
+++ b/core/fpdfdoc/cpdf_metadata.cpp
@@ -8,53 +8,54 @@
#include "core/fpdfapi/parser/cpdf_stream.h"
#include "core/fpdfapi/parser/cpdf_stream_acc.h"
-#include "core/fxcrt/xml/cxml_content.h"
-#include "core/fxcrt/xml/cxml_element.h"
+#include "core/fxcrt/fx_codepage.h"
+#include "core/fxcrt/xml/cfx_xmlelement.h"
+#include "core/fxcrt/xml/cfx_xmlparser.h"
namespace {
-void CheckForSharedFormInternal(CXML_Element* element,
+void CheckForSharedFormInternal(CFX_XMLElement* element,
std::vector<UnsupportedFeature>* unsupported) {
- size_t count = element->CountAttrs();
- for (size_t i = 0; i < count; ++i) {
- ByteString space;
- ByteString name;
- WideString value;
- element->GetAttrByIndex(i, &space, &name, &value);
- if (space != "xmlns" || name != "adhocwf" ||
- value != L"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/") {
+ for (const auto& pair : element->GetAttributes()) {
+ if (pair.first != L"xmlns:adhocwf" ||
+ pair.second != L"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/") {
continue;
}
- CXML_Element* pVersion = element->GetElement("adhocwf", "workflowType", 0);
- if (!pVersion)
- continue;
-
- CXML_Content* pContent = ToContent(pVersion->GetChild(0));
- if (!pContent)
- continue;
-
- switch (pContent->m_Content.GetInteger()) {
- case 0:
- unsupported->push_back(UnsupportedFeature::kDocumentSharedFormEmail);
- break;
- case 1:
- unsupported->push_back(UnsupportedFeature::kDocumentSharedFormAcrobat);
- break;
- case 2:
- unsupported->push_back(
- UnsupportedFeature::kDocumentSharedFormFilesystem);
- break;
+ for (const auto* child = element->GetFirstChild(); child;
+ child = child->GetNextSibling()) {
+ if (child->GetType() != FX_XMLNODE_Element)
+ continue;
+
+ const auto* child_elem = static_cast<const CFX_XMLElement*>(child);
+ if (child_elem->GetName() != L"adhocwf:workflowType")
+ continue;
+
+ switch (child_elem->GetTextData().GetInteger()) {
+ case 0:
+ unsupported->push_back(UnsupportedFeature::kDocumentSharedFormEmail);
+ break;
+ case 1:
+ unsupported->push_back(
+ UnsupportedFeature::kDocumentSharedFormAcrobat);
+ break;
+ case 2:
+ unsupported->push_back(
+ UnsupportedFeature::kDocumentSharedFormFilesystem);
+ break;
+ }
+ // We only care about the first one we find.
+ break;
}
}
- count = element->CountChildren();
- for (size_t i = 0; i < count; ++i) {
- CXML_Element* child = ToElement(element->GetChild(i));
- if (!child)
+ for (auto* child = element->GetFirstChild(); child;
+ child = child->GetNextSibling()) {
+ if (child->GetType() != FX_XMLNODE_Element)
continue;
- CheckForSharedFormInternal(child, unsupported);
+ CheckForSharedFormInternal(static_cast<CFX_XMLElement*>(child),
+ unsupported);
}
}
@@ -70,12 +71,16 @@ std::vector<UnsupportedFeature> CPDF_Metadata::CheckForSharedForm() const {
auto pAcc = pdfium::MakeRetain<CPDF_StreamAcc>(stream_.Get());
pAcc->LoadAllDataFiltered();
- std::unique_ptr<CXML_Element> xml_root =
- CXML_Element::Parse(pAcc->GetData(), pAcc->GetSize());
- if (!xml_root)
+ auto root = pdfium::MakeUnique<CFX_XMLElement>(L"root");
+ auto proxy = pdfium::MakeRetain<CFX_SeekableStreamProxy>(pAcc->GetData(),
+ pAcc->GetSize());
+ proxy->SetCodePage(FX_CODEPAGE_UTF8);
+
+ CFX_XMLParser parser(root.get(), proxy);
+ if (!parser.Parse())
return {};
std::vector<UnsupportedFeature> unsupported;
- CheckForSharedFormInternal(xml_root.get(), &unsupported);
+ CheckForSharedFormInternal(root.get(), &unsupported);
return unsupported;
}
diff --git a/core/fpdfdoc/cpdf_metadata_unittest.cpp b/core/fpdfdoc/cpdf_metadata_unittest.cpp
index 6e6d2f63dd..1a39948461 100644
--- a/core/fpdfdoc/cpdf_metadata_unittest.cpp
+++ b/core/fpdfdoc/cpdf_metadata_unittest.cpp
@@ -10,7 +10,7 @@
TEST(CPDF_MetadataTest, CheckSharedFormEmailAtTopLevel) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<node xmlns:adhocwf=\"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/\">\n"
"<adhocwf:workflowType>0</adhocwf:workflowType>\n"
"<adhocwf:version>1.1</adhocwf:version>\n"
@@ -27,7 +27,7 @@ TEST(CPDF_MetadataTest, CheckSharedFormEmailAtTopLevel) {
TEST(CPDF_MetadataTest, CheckSharedFormAcrobatAtTopLevel) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<node xmlns:adhocwf=\"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/\">\n"
"<adhocwf:workflowType>1</adhocwf:workflowType>\n"
"<adhocwf:version>1.1</adhocwf:version>\n"
@@ -44,7 +44,7 @@ TEST(CPDF_MetadataTest, CheckSharedFormAcrobatAtTopLevel) {
TEST(CPDF_MetadataTest, CheckSharedFormFilesystemAtTopLevel) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<node xmlns:adhocwf=\"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/\">\n"
"<adhocwf:workflowType>2</adhocwf:workflowType>\n"
"<adhocwf:version>1.1</adhocwf:version>\n"
@@ -61,7 +61,7 @@ TEST(CPDF_MetadataTest, CheckSharedFormFilesystemAtTopLevel) {
TEST(CPDF_MetadataTest, CheckSharedFormWithoutWorkflow) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<node xmlns:adhocwf=\"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/\">\n"
"<adhocwf:state>2</adhocwf:state>\n"
"<adhocwf:version>1.1</adhocwf:version>\n"
@@ -77,7 +77,7 @@ TEST(CPDF_MetadataTest, CheckSharedFormWithoutWorkflow) {
TEST(CPDF_MetadataTest, CheckSharedFormAsChild) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<grandparent><parent>\n"
"<node xmlns:adhocwf=\"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/\">\n"
"<adhocwf:workflowType>0</adhocwf:workflowType>\n"
@@ -96,7 +96,7 @@ TEST(CPDF_MetadataTest, CheckSharedFormAsChild) {
TEST(CPDF_MetadataTest, CheckSharedFormAsNoAdhoc) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<node></node>";
CPDF_Stream stream;
@@ -109,7 +109,7 @@ TEST(CPDF_MetadataTest, CheckSharedFormAsNoAdhoc) {
TEST(CPDF_MetadataTest, CheckSharedFormWrongNamespace) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<node xmlns:adhocwf=\"http://ns.adobe.com/AcrobatAdhocWorkflow/2.0/\">\n"
"<adhocwf:workflowType>1</adhocwf:workflowType>\n"
"<adhocwf:version>1.1</adhocwf:version>\n"
@@ -125,7 +125,7 @@ TEST(CPDF_MetadataTest, CheckSharedFormWrongNamespace) {
TEST(CPDF_MetadataTest, CheckSharedFormMultipleErrors) {
const char* data =
- "<?xml charset=utf-8?>\n"
+ "<?xml charset=\"utf-8\"?>\n"
"<grandparent>"
"<parent>\n"
"<node xmlns:adhocwf=\"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/\">\n"
diff --git a/core/fxcrt/xml/cfx_xmlparser_unittest.cpp b/core/fxcrt/xml/cfx_xmlparser_unittest.cpp
index d22925f797..39ddc32987 100644
--- a/core/fxcrt/xml/cfx_xmlparser_unittest.cpp
+++ b/core/fxcrt/xml/cfx_xmlparser_unittest.cpp
@@ -8,8 +8,10 @@
#include "core/fxcrt/cfx_seekablestreamproxy.h"
#include "core/fxcrt/fx_codepage.h"
+#include "core/fxcrt/xml/cfx_xmlnode.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "testing/test_support.h"
+#include "third_party/base/ptr_util.h"
class CFX_XMLTestParser : public CFX_XMLParser {
public: