This page loads immediately in IE, but hangs indefinitely in mozilla. I'm not adept at HTML so I have no idea why.
WFM, 2002061712 Win 98. Reporter, please always specify which Build ID you're using when filing a bug.
wfm with win2k build 20020621..
spins forever on a current CVS, linux - nothing loads. Tried all possible combos of proxies and pipeline settings - no go.
confirmed with linux build 2002062204. works fine with build 2002062021. marking NEW ==> Http
Created attachment 88826 [details] NSPR nsHttp logs for builds 20020620 and 20020622 produced with diff --side-by-side (trivial differences manually ignored)
Finishes displaying in 2002062004 linux, but status bar still says "Transfering data from ...". I get that on quite a few sites... news.bbc.co.uk especially.
Jeremy: regression occurred after your build... marking NEW for real. sniffit showed the document fully transferred and view->source displayed it just fine... ???
this seems to be working with build 20020622 now. A new http log shows only minor differences with build 20020620. if someone can't just look at the logs I attached and know what's going on, this is worksforme.
Still occurs with trunk 2002062304 on Windows ME. View Source does show the code..
regression from bug 26790
*** Bug 154108 has been marked as a duplicate of this bug. ***
got a patch to fix this, will attach asap
Created attachment 89164 [details] [diff] [review] patch to fix this fixes the problem but it's not as beautiful as it should be. What we might end up doing is call ScriptAvailable/ScriptEvaluated more then once if you first insert an empty <script> and then set the src-attribute or give it children. This can never happen during parsing since we always add all attributes/children to a <script> before adding it to the document tree. However it can happen during DOM manipulation. We'll still never evaluate a script more then once though.
bz, got time to review this?
*** Bug 154157 has been marked as a duplicate of this bug. ***
Comment on attachment 89164 [details] [diff] [review] patch to fix r=bzbarsky, but it'd be nice to make the return value from ProcessScriptElement() be a success value that says "script evaluated" or a success value that says "nothing to evaluate". Then we could just check that return value and set mIsEvaluated accordingly....
Comment on attachment 89164 [details] [diff] [review] patch to fix sr=jst, lets get this in and leave the bug open if we want to tweak things more later...
*** Bug 154164 has been marked as a duplicate of this bug. ***
*** Bug 154428 has been marked as a duplicate of this bug. ***
Comment on attachment 89164 [details] [diff] [review] patch to fix this one has landed so all problems should be fixed. However i'm keeping this bug open to fix the suggestion from Boris
Jonas, I'd suggest filing that as a separate bug.
changing this bug to reflect the remaining work needed
Hi, I'm the original submitter. I'm not certain, but I think you believe this bug is resolved, but its not as you can see at: http://www.cbot.com/cbot/quotes/futures_volume_detail/0,1589,12+410+415+9,00.html which still displays incorrectly.
this bug (originally) concerned Mozilla hanging when trying to load a page with a particular <sciprt> syntax. A display problem (which I do see) on a different page is a different bug.
Jonas, are you still planning to do anything in this bug?
No, it appears that we've cleaned things up enough that there's nothing left here to do.