Closed Bug 100638 Opened 23 years ago Closed 23 years ago

WRMB: hourglass never goes away on JS generated content

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 81980
Milestone:
mozilla1.0.1

People

(Reporter: barrowma, Assigned: jst)

References

(
URL
)

Details

(Whiteboard: [bugscape: 6396]) 

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.2) Gecko/20010821
Netscape6/6.1
BuildID:    2001091703

The form on this page (http://www.windsorvineyards.com/label.html) creates a
popup window when you submit.  The HTML content on the popup is created via
document.write()'s.  The hourglass never goes away even though the content is
written completely.

Reproducible: Always
Steps to Reproduce:
1. Go to http://www.windsorvineyards.com/label.html
2. Select some entries in the pulldown lists or enter values in the text fields
(the generated wine label will show the value in the text field iff the pulldown
is not set to some non-empty value)
3. Submit the form by clicking "View Label"

Actual Results:  A popup appears with the correct content, but the hourglass
cursor icon remains, as if content is still loading.

Expected Results:  The cursor should return to its normal state (arrow icon).

The relevant JS is in the following URL: http://www.windsorvineyards.com/vlable.js
Browser, not engine --> DOM Level 0      OS : Win98 --> All
Not sure if this is the right component; please reassign as necessary; thanks.


The "View Label" button is as follow: 

<INPUT TYPE="button" VALUE="View Label" onClick="return go_now()">


function go_now()
{

var total = 0;
var msg1;
var msg2;
var msg3;

var f = window.document.myform;

// CHECKS TO SEE IF PERSONALIZED LABEL OPTION IS SELECTED           

if (f.Selone.options[f.Selone.selectedIndex].value != "none")
{ 
  msg1 = f.Selone.options[f.Selone.selectedIndex].value;
}
else
{      
  msg1 = f.Firstbox.value;
}

if (f.Seltwo.options[f.Seltwo.selectedIndex].value != "none")
{ 
  msg2 = f.Seltwo.options[f.Seltwo.selectedIndex].value;
}
else
{      
  msg2 = f.Secondbox.value;
}

if (f.Selthree.options[f.Selthree.selectedIndex].value != "none")
{ 
  msg3 = f.Selthree.options[f.Selthree.selectedIndex].value;
}
else
{      
  msg3 = f.Thirdbox.value;
}


if (f.Italics.checked)
{
var d = window.open("", "MyWin", "resizable,width=500,height=500").document;

d.write('<html><head><title>wine Label</title></head>');
d.write('<body bgcolor="#ffffff" link="#000000" marginwidth="0" topmargin="0" 
leftmargin="0" marginheight="0" onblur="window.close()">');
d.write('<br><br>');
d.write('<center>');
d.write('<table background="images/labelbg.gif" border="1" cellspacing=0 
cellpadding=0>');
d.write('<tr><td align="center">');
d.write('<table border="0" width="394" cellspacing=0 cellpadding=0>');
d.write('<tr><td align="center">');
d.write('<img src="images/bstrip.gif" width="394" height="10"><br><img 
src="images/toplabel.gif" width="244" height="136">');
d.write('</td></tr>');
d.write('<tr><td align="center"><br>');
d.write('<font size="3"><i>'+msg1);
d.write('</i></font>');
d.write('</td></tr>');
d.write('<tr><td align="center">');
d.write('<font size=3><i>'+msg2);
d.write('</i></font>');
d.write('</td></tr>');
d.write('<tr><td align="center">');
d.write('<font size=3><i>'+msg3);
d.write('</i></font>');
d.write('</td></tr>');
d.write('<tr><td align="center">');
d.write('<br><img src="images/labelbot.gif" width="226" height="82">');
d.write('</td></tr></table>');
d.write('</td></tr></table>');
d.write('<br><br>');
d.write('<table border="2" bgcolor="#cccccc" cellspacing=0 cellpadding=2>');
d.write('<tr><td align="center" valign="top">');
d.write('<a href="javascript:window.close()"><font size=3>Click Here To Revise 
Or Make A New Label</font></a>');
d.write('</td></tr></table>');
d.write('<br>');
d.write('</center>');
d.write('</body>');
d.write('</html>');
d.close;

return true;
}

else
{
var d = window.open("", "MyWin", "resizable,width=500,height=500").document;

d.write('<html><head><title>wine Label</title></head>');
d.write('<body bgcolor="#ffffff" link="#000000" marginwidth="0" topmargin="0" 
leftmargin="0" marginheight="0" onblur="window.close()">');
d.write('<br><br>');
d.write('<center>');
d.write('<table background="images/labelbg.gif" border="1" cellspacing=0 
cellpadding=0>');
d.write('<tr><td align="center">');
d.write('<table border="0" width="394" cellspacing=0 cellpadding=0>');
d.write('<tr><td align="center">');
d.write('<img src="images/bstrip.gif" width="394" height="10"><br><img 
src="images/toplabel.gif" width="244" height="136">');
d.write('</td></tr>');
d.write('<tr><td align="center"><br>');
d.write('<font size=3>'+msg1);
d.write('</font>');
d.write('</td></tr>');
d.write('<tr><td align="center">');
d.write('<font size=3>'+msg2);
d.write('</font>');
d.write('</td></tr>');
d.write('<tr><td align="center">');
d.write('<font size=3>'+msg3);
d.write('</font>');
d.write('</td></tr>');
d.write('<tr><td align="center">');
d.write('<br><img src="images/labelbot.gif" width="226" height="82">');
d.write('</td></tr></table>');
d.write('</td></tr></table>');
d.write('<br><br>');
d.write('<table border="2" bgcolor="#cccccc"cellspacing=0 cellpadding=2>');
d.write('<tr><td align="center">');
d.write('<a href="javascript:window.close()"><font size=3>Click Here To Revise 
Or Make A New Label</font></a>');
d.write('</td></tr>');
d.write('</table>');
d.write('<br>');
d.write('</center>');
d.write('</body>');
d.write('</html>');
d.close;

return true;

 }    
}
Assignee: rogerl → jst
Component: Javascript Engine → DOM Level 0
OS: Windows 98 → All
QA Contact: pschwartau → desale
  d.write(...);
  ...
  d.close;

that should be d.close();, i.e. missing parens. That's why we never finish
loading this that page. There's not much we can do about this, and given that
this doesn't actually make the site unusable I'm marking this as WONTFIX, if you
disagree, please reopen and reassign to evangelism.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
Marking Verified Wontfix -
Status: RESOLVED → VERIFIED
If there is no JS left to execute, shouldn't the hourglass go away, regardless
of the incorrect syntax of the final statement?
Status: VERIFIED → REOPENED
Resolution: WONTFIX → ---
Keywords: topembed
adding WRMB
Summary: hourglass never goes away on JS generated content → WRMB: hourglass never goes away on JS generated content
adding bugscape ref
Whiteboard: [bugscape: 6396]
We don't know that there is no more JS left to execute, we have no idea who has
references to the document, so we can't check if there's more JS about to
execute or not. We don't have a solution for this, we might be able to fix this
with fireing off a timer when document.write() is called, and then after some
time if no more calls have been made to document.write() and document.close()
was not called we could close the document, or something like that. I don't see
why this would get high priority tho, are there top 100 sites that show this
problem?
Status: REOPENED → ASSIGNED
Target Milestone: --- → mozilla1.0
Blocks: 64833
removing topembed
Keywords: topembed
Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 
(you can query for this string to delete spam or retrieve the list of bugs I've 
moved)
Target Milestone: mozilla1.0 → mozilla1.0.1

*** This bug has been marked as a duplicate of 81980 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.