Move get-html into core/dom

src/get-html.js → src/core/dom/get-html.js

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import {isElement} from './core/types';
+import {isElement, isString} from '../types';
 
 /** @type {!Array<string>} */
 const excludedTags = ['script', 'style'];
@@ -76,14 +76,12 @@ export function getHtml(win, selector, attrs) {
  */
 function appendToResult(node, attrs, result) {
   const stack = [node];
-  const allowedAttrs = attrs.filter((attr) => {
-    return allowedAttributes.includes(attr);
-  });
+  const allowedAttrs = attrs.filter((attr) => allowedAttributes.includes(attr));
 
   while (stack.length > 0) {
     node = stack.pop();
 
-    if (typeof node === 'string') {
+    if (isString(node)) {
       result.push(node);
     } else if (node.nodeType === Node.TEXT_NODE) {
       result.push(node.textContent);
@@ -99,7 +97,7 @@ function appendToResult(node, attrs, result) {
 }
 
 /**
- *
+ * Returns true for allowed AMP tags and non-AMP tags except <script>/<style>
  * @param {!Element} node
  * @return {boolean}
  */
@@ -114,15 +112,15 @@ function isApplicableNode(node) {
 }
 
 /**
- *
+ * Constructs an open-tag with the provided attributes.
  * @param {!Element} node
  * @param {!Array<string>} attrs
  * @param {Array<string>} result
  */
 function appendOpenTag(node, attrs, result) {
   result.push(`<${node.tagName.toLowerCase()}`);
 
-  attrs.forEach(function (attr) {
+  attrs.forEach((attr) => {
     if (node.hasAttribute(attr)) {
       result.push(` ${attr}="${node.getAttribute(attr)}"`);
     }

test/unit/test-get-html.js → test/unit/core/dom/test-get-html.js

@@ -26,9 +26,9 @@ const template = `<div id="wrapper">
 </div>
 `;
 
-import {getHtml} from '../../src/get-html';
+import {getHtml} from '../../../../src/core/dom/get-html';
 
-describes.sandboxed('getHtml', {}, () => {
+describes.sandboxed('DOM - getHtml', {}, () => {
   let element;
 
   beforeEach(() => {