38574 - Numerous assertions and eventual crash on NT trying to submit a form to anywhere

Status: VERIFIED WORKSFORME

(Core :: Web Painting, defect, P1)

Description

[ruslan]

Numerous assertions and eventual crash on NT trying to submit a form anything;

void nsViewManager2::GetDrawingSurfaceSize(nsRect& aRequestedSize, nsRect& 
aNewSize) const
{ 
  if (PR_FALSE == CalculateDiscreteSurfaceSize(aRequestedSize, aNewSize)) {
    NS_ASSERTION(PR_FALSE, "CalculateDiscreteSize failed");

Was working fine on Friday.

Comment 1

[Asa Dotzler [:asa]]

um, Form Submission maybe.

Comment 2

[rods (gone)]

reassigning

Comment 3

[Eric Pollmann]

I see many assertions as well.  However, I don't think this has anything to do 
with form submission:

1) The form submit goes through with no assertions or crash in viewer
2) If you shrink MSVC down so that it does not intersect any area of the mozilla 
window, then continue past the view manager assertions, the form submit goes 
through with no crash
3) The assertion is in the view manager.

I also notice that the security dialog box, which now comes up but did not come 
up on friday, also only comes up in Moz and not viewer - this might have 
something to do with the problem too?

Comment 4

[Doug Turner (:dougt)]

hmm, Try doing this:

open up bin/chrome/navigator/locale/security.properties
truncate |PostToInsecure| and |PostToInsecureFromInsecure| to only a few words.

Does this fix the problem?

Comment 5

[ruslan]

Yes this fixed it.

Comment 6

[ruslan]

Though I can see assertions like that also by dragging other dialog boxes over 
the form

Comment 7

[Doug Turner (:dougt)]

so, the bug is alerts can not take too much text.  We need to reassign to ben or 
someone that owns these dialogs.  This used to work.

Comment 8

[Eric Pollmann]

Reassigning per above suggestion...

Comment 9

[leger]

Putting on [dogfood+][nsbeta2-] radar.  Please fix ASAP.

Comment 10

[Doug Turner (:dougt)]

ben, let me know if you want me to truncate my text.

Comment 11

[don]

Ben, do the dialogs need to be fixed or should Doug Turner just truncate the
text?

Comment 12

[leger]

Adding crash keyword

Comment 13

[Ben Goodger (use ben at mozilla dot org for email)]

common dialogs should be able to take an infinite amount of text and Just Deal.

truncate if you like, dougt. I'll look at this issue in more detail when I 
inspect the cdlg code. 

Comment 14

[Doug Turner (:dougt)]

ben, than this is a bug in your cdlg code.  I want to be able to use the same 
text for these warning as Netscape 4.71 did.  

Back to you...

A workaround is to open the Security Advisor and disable all notifications under 
the Applications tab.

Comment 15

[Ben Goodger (use ben at mozilla dot org for email)]

I don't see this (Windows 2000 ~ NT) submitting the zip code form on the given 
page...

Ruslan, is there anything else I should be doing? 

The common dialog code is mine via inheritance. I have not had a chance to take 
a good look at it yet by from initial glances it seems to me that it requires a 
complete rewrite, and whether or not that would affect this I don't know. This 
could take some time. Just letting y'all know. 

Comment 16

[Doug Turner (:dougt)]

this does not happen to me anymore (today's and yesterday's build)  marking as 
WORKSFORME.

Comment 17

[ckritzer (gone)]

Marking VERIFED WORKSFORME on:
- MacOS9 2000-06-01-20-M16 Commercial Build
- Linux6 2000-06-01-20-M16 Commercial Build
- Win98  2000-06-01-21-M16 Commercial Build
- WinNT  2000-06-01-21-M16 Commercial Build