34669 - Page with script-generated nested fieldsets has many assertions and prints only top portion

Status: RESOLVED INVALID

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

Description

[shrirang khanzode]

Using windows build:

1. Install and launch the browser
2. Go to the page above
3. Print the page 
4. Observe that only the top portion of the page with the file name and 
"Exceptions: There are 46 exception tests" prints. Other pages do not print at 
all.

Thanks to Chris for finding this problem.

Comment 1

[dcone (gone)]

I get a ton of asserts in the nsFrame code.. just loading this page.  I can 
understand why we don't print it.  Looks like nsCSSFrameConstructor is causing 
the asserts.  Mark can you look into this.

Comment 2

[Marc Attinasi]

We are getting a boatload of assertions stemming from the javascript execution 
and that is probably what is causing the print to fail. A simple HTML testcase 
with nested FIELDSETs prints just fine (I'll attach it).

Javascript engineer should look at this to determine why there are so many 
errors / assertions executing this script. I cannot make sense out of the 
script, however I see that some of the assertions are complaining about invalid 
document insertions and null pointers.

Comment 3

[Marc Attinasi]

Attachment: Simple page (no script) showing that nested fieldsets print fine.

Comment 4

[rogerl (gone)]

None of the failures are related to Javascript errors. The test case provokes a 
ton of assertions from nsDocument/nsFrame, whether these are correct i have no 
way of knowing...

Comment 5

[troy]

I see an assert in the nsDocument::ReplaceChild() code. Here's the stack trace:

nsDebug::Assertion(const char * 0x018b6190, const char * 0x018b615c, const char 
* 0x018b6130, int 2446) line 191 + 13 bytes
nsDocument::ReplaceChild(nsDocument * const 0x00dc60a4, nsIDOMNode * 0x00000000, 
nsIDOMNode * 0x00000000, nsIDOMNode * * 0x0088d094) line 2446 + 38 bytes
nsHTMLDocument::ReplaceChild(nsHTMLDocument * const 0x00dc60a4, nsIDOMNode * 
0x00000000, nsIDOMNode * 0x00000000, nsIDOMNode * * 0x0088d094) line 1310
NodeReplaceChild(JSContext * 0x034dae90, JSObject * 0x00bae048, unsigned int 2, 
long * 0x00b4f838, long * 0x0088d144) line 472 + 34 bytes
js_Invoke(JSContext * 0x034dae90, unsigned int 2, unsigned int 0) line 686 + 23 
bytes
js_Interpret(JSContext * 0x034dae90, long * 0x0088db08) line 2483 + 15 bytes
js_Execute(JSContext * 0x034dae90, JSObject * 0x00bae4c8, JSScript * 0x035914c0, 
JSFunction * 0x0359c9a0, JSStackFrame * 0x0088e3cc, unsigned int 2, long * 
0x0088db08) line 857 + 13 bytes
obj_eval(JSContext * 0x034dae90, JSObject * 0x00acfa90, unsigned int 1, long * 
0x00b4f810, long * 0x0088db08) line 846 + 47 bytes
js_Invoke(JSContext * 0x034dae90, unsigned int 1, unsigned int 0) line 686 + 23 
bytes
js_Interpret(JSContext * 0x034dae90, long * 0x0088e430) line 2483 + 15 bytes
js_Invoke(JSContext * 0x034dae90, unsigned int 0, unsigned int 0) line 702 + 13 
bytes
js_Interpret(JSContext * 0x034dae90, long * 0x0088ed90) line 2483 + 15 bytes
js_Execute(JSContext * 0x034dae90, JSObject * 0x00acfa90, JSScript * 0x0359d460, 
JSFunction * 0x00000000, JSStackFrame * 0x00000000, unsigned int 0, long * 
0x0088ed90) line 857 + 13 bytes
JS_EvaluateUCScriptForPrincipals(JSContext * 0x034dae90, JSObject * 0x00acfa90, 
JSPrincipals * 0x03564530, const unsigned short * 0x00b5e264, unsigned int 7124, 
const char * 0x0356ef40, unsigned int 44, long * 0x0088ed90) line 2747 + 27 
bytes
nsJSContext::EvaluateString(nsJSContext * const 0x034da090, const nsString & 
{...}, void * 0x00acfa90, nsIPrincipal * 0x0356452c, const char * 0x0356ef40, 
unsigned int 44, const char * 0x0063b468, nsString & {...}, int * 0x0088edec) 
line 463 + 55 bytes
HTMLContentSink::EvaluateScript(nsString & {...}, nsIURI * 0x035deaf0, int 44, 
const char * 0x0063b468) line 4326
HTMLContentSink::ProcessSCRIPTTag(const nsIParserNode & {...}) line 4636
HTMLContentSink::AddLeaf(HTMLContentSink * const 0x00dc6a60, const nsIParserNode 
& {...}) line 2981 + 12 bytes
CNavDTD::AddLeaf(const nsIParserNode * 0x03568170) line 3240 + 22 bytes
CNavDTD::HandleScriptToken(const nsIParserNode * 0x03568170) line 1824 + 12 
bytes
CNavDTD::OpenContainer(const nsIParserNode * 0x03568170, nsHTMLTag 
eHTMLTag_script, int 1, nsEntryStack * 0x00000000) line 2892 + 12 bytes
CNavDTD::HandleDefaultStartToken(CToken * 0x035550e0, nsHTMLTag eHTMLTag_script, 
nsIParserNode * 0x03568170) line 1025 + 20 bytes
CNavDTD::HandleStartToken(CToken * 0x035550e0) line 1363 + 22 bytes
CNavDTD::HandleToken(CNavDTD * const 0x0355ad80, CToken * 0x035588b0, nsIParser 
* 0x00dc4c90) line 709 + 12 bytes
CNavDTD::BuildModel(CNavDTD * const 0x0355ad80, nsIParser * 0x00dc4c90, 
nsITokenizer * 0x0355a8c0, nsITokenObserver * 0x00000000, nsIContentSink * 
0x00dc6a60) line 447 + 20 bytes
nsParser::BuildModel() line 1230 + 34 bytes
nsParser::ResumeParse(int 1, int 0) line 1114 + 11 bytes
nsParser::EnableParser(int 1) line 757 + 15 bytes
HTMLContentSink::ResumeParsing() line 4259 + 19 bytes
HTMLContentSink::OnStreamComplete(HTMLContentSink * const 0x00dc6a64, 
nsIStreamLoader * 0x0355f830, nsISupports * 0x00000000, unsigned int 0, unsigned 
int 17090, const char * 0x00b56230) line 4456 + 11 bytes
nsStreamLoader::OnStopRequest(nsStreamLoader * const 0x0355f834, nsIChannel * 
0x0355f680, nsISupports * 0x00000000, unsigned int 0, const unsigned short * 
0x00000000) line 120 + 78 bytes
nsHTTPFinalListener::OnStopRequest(nsHTTPFinalListener * const 0x0355f7d0, 
nsIChannel * 0x0355f680, nsISupports * 0x00000000, unsigned int 0, const 
unsigned short * 0x00000000) line 1115 + 42 bytes
nsHTTPChannel::ResponseCompleted(nsIStreamListener * 0x0355f7d0, unsigned int 0, 
const unsigned short * 0x00000000) line 1586 + 36 bytes
nsHTTPServerListener::OnStopRequest(nsHTTPServerListener * const 0x03567770, 
nsIChannel * 0x0355f334, nsISupports * 0x0355f680, unsigned int 0, const 
unsigned short * 0x00000000) line 659
nsOnStopRequestEvent::HandleEvent(nsOnStopRequestEvent * const 0x035645f0) line 
307
nsStreamListenerEvent::HandlePLEvent(PLEvent * 0x035645a0) line 97 + 12 bytes
PL_HandleEvent(PLEvent * 0x035645a0) line 575 + 10 bytes
PL_ProcessPendingEvents(PLEventQueue * 0x00db0b80) line 520 + 9 bytes
_md_EventReceiverProc(HWND__ * 0x00000f80, unsigned int 53401, unsigned int 0, 
long 14355328) line 1030 + 9 bytes
KERNEL32! bff7363b()

Comment 6

[troy]

Reassigning to jst

Comment 7

[Johnny Stenback (:jst)]

Please make the original sample accessible from outside the Netscape fw.

Comment 8

[ckritzer (gone)]

Testcases will be @ 
http://people.netscape.com/ckritzer/mozilla/bugzilla/index.html after the 
internal server push tonight ~ midnight.

Comment 9

[Johnny Stenback (:jst)]

This bug has been marked "future" because the original netscape engineer

workingon this is over-burdened. If you feel this is an error, that you or

another known resource will be working on this bug,or if it blocks your work in

some way -- please attach your concern to the bug for reconsideration.

Comment 10

[Fabian Guisset]

I can't seem to open the URL mentioned in the above URL field. Anything I can do
to reproduce?

Comment 11

[Jesse Ruderman]

Fixing summary to not end with "loads of ass" when truncated at 60 chars.

Comment 12

[Johnny Stenback (:jst)]

ROFL :-)

Comment 13

[Johnny Stenback (:jst)]

Mass-reassigning bugs to [e-mail]

Comment 14

[Fabian Guisset]

The URL in the URL field does not work. Can't do anything about this as it is.
Invalid?

Comment 15

[Gérard Talbot]

Attachment: Script page creating dynamically nested fieldsets which print fine

Load the attachment, click the button and print the page.

WORKSFORME

Mozilla 1.6 (build 20040113) under XP Pro SP1.

It also worked for me even without creating+inserting <legend>.

Comment 16

[Gérard Talbot]

The testcase of attachment 142966 [details] WORKSFORME with Mozilla 1.7b 2004031309.

Comment 17

[Boris Zbarsky [:bzbarsky]]

Marking invalid, since I can't reproduce and the testcases and such are gone. 
"make it available outside netscape" should have been clarified to "attach it to
the friggin' bug", perhaps....