Added code scoping fix. General codestyle fixes and optimizations

autoHeightWebView/utils.js

@@ -13,9 +13,6 @@ observer.observe(document, {
 
 const updateSizeWithMessage = element =>
   `
-  var updateSizeInterval = null;
-  var height = 0;
-
   var lastHeight = 0;
   var heightTheSameTimes = 0;
   var maxHeightTheSameTimes = 5;
@@ -23,13 +20,17 @@ const updateSizeWithMessage = element =>
   var forceRefreshTimeout;
 
   function updateSize(event) {
-    if (!window.hasOwnProperty('ReactNativeWebView') || !window.ReactNativeWebView.hasOwnProperty('postMessage')) {
-      !updateSizeInterval && (updateSizeInterval = setInterval(updateSize, 200));
+    if (
+      !window.hasOwnProperty('ReactNativeWebView') || 
+      !window.ReactNativeWebView.hasOwnProperty('postMessage')
+    ) {
+      setTimeout(updateSize, 200);
       return;
     }
-    clearInterval(updateSizeInterval)
-    height = ${element}.offsetHeight || window.innerHeight;
-    width = ${element}.offsetWidth || window.innerWidth;
+    
+    var height = ${element}.offsetHeight || window.innerHeight;
+    var width = ${element}.offsetWidth || window.innerWidth;
+
     window.ReactNativeWebView.postMessage(JSON.stringify({ width: width, height: height }));
 
     // Make additional height checks (required to fix issues wit twitter embeds)
@@ -71,7 +72,6 @@ const getBaseScript = ({ style, zoomable }) =>
     }
     document.body.appendChild(wrapper);
   }
-  var width = ${getWidth(style)};
   ${updateSizeWithMessage('wrapper')}
   window.addEventListener('load', updateSize);
   window.addEventListener('resize', updateSize);
@@ -84,13 +84,13 @@ const appendFilesToHead = ({ files, script }) =>
   files.reduceRight((combinedScript, file) => {
     const { rel, type, href } = file;
     return `
-          var link  = document.createElement('link');
-          link.rel  = '${rel}';
-          link.type = '${type}';
-          link.href = '${href}';
-          document.head.appendChild(link);
-          ${combinedScript}
-        `;
+      var link  = document.createElement('link');
+      link.rel  = '${rel}';
+      link.type = '${type}';
+      link.href = '${href}';
+      document.head.appendChild(link);
+      ${combinedScript}
+    `;
   }, script);
 
 const screenWidth = Dimensions.get('window').width;
@@ -107,17 +107,20 @@ const appendStylesToHead = ({ style, script }) => {
   // Escape any single quotes or newlines in the CSS with .replace()
   const escaped = currentStyles.replace(/\'/g, "\\'").replace(/\n/g, '\\n');
   return `
-          var styleElement = document.createElement('style');
-          styleElement.innerHTML = '${escaped}';
-          document.head.appendChild(styleElement);
-          ${script}
-        `;
+    var styleElement = document.createElement('style');
+    styleElement.innerHTML = '${escaped}';
+    document.head.appendChild(styleElement);
+    ${script}
+  `;
 };
 
 const getInjectedSource = ({ html, script }) => `
 ${html}
 <script>
-${script}
+// prevents code colissions with global scope
+(() => {
+  ${script}
+})();
 </script>
 `;