Change CFX_XML Save to take a write stream

This CL changes CFX_XML to use an IFX_SeekableWriteStream instead of the more
generic IFX_SeekableStream.

Change-Id: I6e4def380c43eca755d91ad5cb6146c2dfdaee10
Reviewed-on: https://pdfium-review.googlesource.com/30877
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>

1 files changed, 4 insertions, 6 deletions

diff --git a/core/fpdfdoc/cpdf_metadata.cpp b/core/fpdfdoc/cpdf_metadata.cpp
index 161fc93edd..c928952dca 100644
--- a/core/fpdfdoc/cpdf_metadata.cpp
+++ b/core/fpdfdoc/cpdf_metadata.cpp
@@ -19,12 +19,11 @@ void CheckForSharedFormInternal(CFX_XMLElement* element,
                                 std::vector<UnsupportedFeature>* unsupported) {
   WideString attr = element->GetAttribute(L"xmlns:adhocwf");
   if (attr == L"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/") {
-    for (const auto* child = element->GetFirstChild(); child;
-         child = child->GetNextSibling()) {
+    for (const auto& child : *element) {
       if (child->GetType() != FX_XMLNODE_Element)
         continue;
 
-      const auto* child_elem = static_cast<const CFX_XMLElement*>(child);
+      const auto* child_elem = static_cast<const CFX_XMLElement*>(child.get());
       if (child_elem->GetName() != L"adhocwf:workflowType")
         continue;
 
@@ -46,12 +45,11 @@ void CheckForSharedFormInternal(CFX_XMLElement* element,
     }
   }
 
-  for (auto* child = element->GetFirstChild(); child;
-       child = child->GetNextSibling()) {
+  for (const auto& child : *element) {
     if (child->GetType() != FX_XMLNODE_Element)
       continue;
 
-    CheckForSharedFormInternal(static_cast<CFX_XMLElement*>(child),
+    CheckForSharedFormInternal(static_cast<CFX_XMLElement*>(child.get()),
                                unsupported);
   }
 }