fixed bug where non-JSON text/html pages were polluted

Author: roth-michael

extension/src/json-viewer/check-if-json.js

@@ -1,4 +1,5 @@
 var extractJSON = require('./extract-json');
+var bodyModified = false;
 
 function allTextNodes(nodes) {
   return !Object.keys(nodes).some(function(key) {
@@ -39,12 +40,21 @@ function getPreWithSource() {
     pre.textContent = textContent;
     document.body.removeChild(childNode);
     document.body.appendChild(pre);
+    bodyModified = true;
     return pre;
   }
 
   return null
 }
 
+function restoreNonJSONBody() {
+  var artificialPre = document.body.lastChild;
+  var removedChildNode = document.createElement("text");
+  removedChildNode.textContent = artificialPre.textContent;
+  document.body.insertBefore(removedChildNode, document.body.firstChild);
+  document.body.removeChild(artificialPre);
+}
+
 function isJSON(jsonStr) {
   var str = jsonStr;
   if (!str || str.length === 0) {
@@ -69,6 +79,8 @@ function checkIfJson(sucessCallback, element) {
     (isJSON(pre.textContent) || isJSONP(pre.textContent))) {
 
     sucessCallback(pre);
+  } else if (bodyModified) {
+    restoreNonJSONBody();
   }
 }