Cleanup CFX_XMLDoc::SaveXMLNode

The CFX_XMLDoc::SaveXMLNode method is almost an exact copy of the
CFX_XMLNode::SaveXMLNode. This CL removes the XMLDoc variant and calls
the XMLNode method directly.

This Removes the need to pass the CXFA_DocumentParser into CXFA_Document
and we can instead pass in the CXFA_FFNotify object directly.

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

1 files changed, 2 insertions, 15 deletions

diff --git a/xfa/fxfa/parser/cxfa_node_unittest.cpp b/xfa/fxfa/parser/cxfa_node_unittest.cpp
index 7c35e5d989..81e21890df 100644
--- a/xfa/fxfa/parser/cxfa_node_unittest.cpp
+++ b/xfa/fxfa/parser/cxfa_node_unittest.cpp
@@ -7,19 +7,9 @@
 #include "testing/test_support.h"
 #include "third_party/base/ptr_util.h"
 #include "xfa/fxfa/parser/cxfa_document.h"
-#include "xfa/fxfa/parser/cxfa_document_parser.h"
 
 namespace {
 
-class CXFA_DocumentMock : public CXFA_Document {
- public:
-  explicit CXFA_DocumentMock(CXFA_DocumentParser* parser)
-      : CXFA_Document(parser) {}
-  ~CXFA_DocumentMock() override = default;
-
-  CXFA_FFNotify* GetNotify() const override { return nullptr; }
-};
-
 class TestNode : public CXFA_Node {
  public:
   explicit TestNode(CXFA_Document* doc)
@@ -40,23 +30,20 @@ class TestNode : public CXFA_Node {
 class CXFANodeTest : public testing::Test {
  public:
   void SetUp() override {
-    doc_parser_ = pdfium::MakeUnique<CXFA_DocumentParser>(nullptr);
-    doc_ = pdfium::MakeUnique<CXFA_DocumentMock>(doc_parser_.get());
+    doc_ = pdfium::MakeUnique<CXFA_Document>(nullptr);
     node_ = pdfium::MakeUnique<TestNode>(doc_.get());
   }
 
   void TearDown() override {
     node_ = nullptr;
     doc_ = nullptr;
-    doc_parser_ = nullptr;
   }
 
   CXFA_Document* GetDoc() const { return doc_.get(); }
   CXFA_Node* GetNode() const { return node_.get(); }
 
  private:
-  std::unique_ptr<CXFA_DocumentParser> doc_parser_;
-  std::unique_ptr<CXFA_DocumentMock> doc_;
+  std::unique_ptr<CXFA_Document> doc_;
   std::unique_ptr<TestNode> node_;
 };