Move DocumentXMLTreeViewer off of private script

third_party/WebKit/Source/core/xml/DocumentXMLTreeViewer.h

Index: third_party/WebKit/Source/core/xml/DocumentXMLTreeViewer.h
diff --git a/third_party/WebKit/Source/core/xml/DocumentXMLTreeViewer.h b/third_party/WebKit/Source/core/xml/DocumentXMLTreeViewer.h
new file mode 100644
index 0000000000000000000000000000000000000000..d976d26fc7d572237ecad71abd7755f7fda2c200
--- /dev/null
+++ b/third_party/WebKit/Source/core/xml/DocumentXMLTreeViewer.h
@@ -0,0 +1,32 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "platform/heap/Heap.h"
+#include "platform/heap/Member.h"
+#include "platform/heap/Visitor.h"
+#include "wtf/Noncopyable.h"
+
+#ifndef DocumentXMLTreeViewer_h
+#define DocumentXMLTreeViewer_h
+
+namespace blink {
+
+class Document;
+
+class DocumentXMLTreeViewer final
+    : public GarbageCollected<DocumentXMLTreeViewer> {

[jbroman, 2016/12/10 20:35:31]

No need to make this object garbage-collected, since you only ever use it on the
stack (it can be marked STACK_ALLOCATED instead; see wtf/Allocator.h).

But further, since this object doesn't really maintain any state, I think this
could just be a function in the blink namespace with no class at all.

void transformDocumentToXMLTreeView(Document*);

[adithyas, 2016/12/12 16:13:50]

Yup, that makes sense. I've changed it to a function.

+ public:
+  WTF_MAKE_NONCOPYABLE(DocumentXMLTreeViewer);
+  DECLARE_TRACE();
+
+  explicit DocumentXMLTreeViewer(Document* document) : m_document(document) {}
+  void transformDocumentToTreeView();
+
+ private:
+  Member<Document> m_document;
+};
+
+}  // namespace blink
+
+#endif