Move DocumentXMLTreeViewer off of private script

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

+// 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