Recent replaceAssets change can cause infinite loop in Chrome about htmlpreview.github.com HOT 4 CLOSED

The last change allows pages like Impress.js to work, by forcing the right order of <script>s. Before this, all external <script src>s were executed at the end, after executing inline <script>Impress.init()</script>, which caused an error of Impress being undefined. Moving inline <script>s to the end as well restores the right order of JS.
However, I will look closer at this issue and make some fix soon.

from htmlpreview.github.com.

How about simply storing script.length into another variable before starting the loop, and then compare against that in the for loop? That way it will only iterate over the original size of the array, regardless of whether the array grows during loop iteration.

from htmlpreview.github.com.

There's much simplier solution - replacing getElementsByTagName with querySelectorAll, which returns a static NodeList.

from htmlpreview.github.com.

Looks good now in Chrome, thanks!

from htmlpreview.github.com.