157632 - window.print() not working (trunk only)

Status: VERIFIED FIXED

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

Description

[Tom Everingham]

Overview Description:  Javascript url's do not appear to be working from within
href's.   

Steps to Reproduce:
1.)  Go to URL listed (internal only) 
2.)  click on the 'click here' link

Actual Results:  nothing happens 

Expected Results:  the text of the main window should be sent to your printer

Build Date & Platform Bug Found:  trunk builds - 07/12/2002  winNT4, linux rh6,
mac osX

Additional Information:  
- works on branch
- using the javascript alert function doesn't work from within href
- examples of this can be found at http://www.mapquest.com and http://www.401k.com

Comment 1

[Tom Everingham]

Attachment: testcase

Comment 2

[Johnny Stenback (:jst)]

This has nothing to do with javascript: URL's, window.print() seems broken. If I
replace the javascript: URL with javascript:alert("foo"); it works. Over to rods
who's the owner of window.print() and everything it does...

Comment 3

[rods (gone)]

Attachment: patch

My fault.. how I forgot to test this testcase is a mystery.
As it turns out clicking on a link makes the document "busy" with a "before
page load" flag. (who would have thought)

This check to make sure the document is busy with an actual page load.

Comment 4

[rods (gone)]

Attachment: testcase (both onload and with a link and a button)

I mis that test because I als check with an input button and forget about the
link case.

Comment 5

[dcone (gone)]

Comment on attachment 91502 [details] [diff] [review]
patch

r=dcone

Comment 6

[Johnny Stenback (:jst)]

Comment on attachment 91502 [details] [diff] [review]
patch

sr=jst

Comment 7

[Scott Collins]

Comment on attachment 91502 [details] [diff] [review]
patch

a=scc for checkin to the mozilla trunk

Comment 8

[rods (gone)]

fixed

Comment 9

[rods (gone)]

fixed again

Comment 10

[Tom Everingham]

this is working for me now on 07/26/02 trunk builds

verified