26335 - mac: unable to use Safe Form Fill, hangs browser

Status: VERIFIED WORKSFORME

(Toolkit :: Form Manager, defect, P3)

Description

[sairuh (rarely reading bugmail)]

found this using the 2000020215 non-comm bits on mac.

to repro,
0. make sure you're Wallet isn't empty ;-)
1. go to a form to fill out, eg, the url above.
2. go to Tasks > My Wallet > Safe Form Fill

result: the Safe Form Fill dialog is completely blank and unresponsive to
mouseclicks. i needed to use MacsBug to get out (lemme know if there's an easier
way :-).

Comment 1

[sairuh (rarely reading bugmail)]

nominating for beta1 and also looks like a crasher.

Comment 2

[leger]

Putting on PDT+ radar for beta1.

Comment 3

[leger]

sariuh, can you get a stack trace from Talkback?

Comment 4

[Stephen P. Morse]

Reassigning to ben

Comment 5

[sairuh (rarely reading bugmail)]

Attachment: sparse stack trace

Comment 6

[sairuh (rarely reading bugmail)]

i haven't gotten a talkback report at all on this --tb didn't pop up when this 
hanging occurred... gonna check w/cyeh to see if i can somehow 'force' talkback 
to generate something on the mac when it hangs...

Comment 7

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

don, do not wish to take wallet bugs. 

Comment 8

[don]

dp, come and talk to me about this ...

Comment 9

[Suresh Duddi (gone)]

Dont know when why this is happening. Steve if you can find out a little more on
where it is hanging, we can make sure there isn't a generic problem somewhere
else that this will resolve. Scc might be able to help you since this is a mac
issue.

Since wallet isn't in the beta list, I am going to downgrade this from PDT+.

Comment 10

[Stephen P. Morse]

Scott, per dp's suggestion, can you take a look at this and tell me what's 
causing the hang.  It's a mac-only problem.

Comment 11

[leger]

Putting on PDT- radar for beta1.

Comment 12

[sairuh (rarely reading bugmail)]

hokay, this what i see using the comm bits from 2000-02-11-12, following the 
original recipe:

the Safe Form Fill dialog *does* appear. so, i click the OK button. the dialog 
goes away... then seamonkey hangs... i click the Back button to see if i can 
simply go to another page...then seamonkey crashes.

now that Talkback is working (yay!), i can add the stack trace info [soon].

Comment 13

[sairuh (rarely reading bugmail)]

talkback info doesn't have much, alas. i'll attach the Macsbug log that i got,
which has more.

http://cyclone/reports/incidenttemplate.CFM?reportID=124&style=0&tc=24&cp=1&ck1=SUser+email+address&cd1=%25sairuh%40netscape%2Ecom%25&co1=like&bbid=5363529


 Trigger Type:  Program Crash 

 Trigger Reason:  PowerPC unmapped memory exception 

 Call Stack:    (Signature = 0x4bf6d754 c6d89041) 
     
  0x4bf6d754 
                        
     
  0x00016b44 
                        
     
  0x00016cc4 
                        
     
  0x00006370 
                        
     
  0x000010dc 
                        
     
  CODE.10 
                        
     
  XPTC_InvokeByIndex() 
                       [xptcinvoke_mac.cpp, line 129]
     
  0x0001e810 
                        
     
  0x0001fe3c 
                        
     
  js_Invoke() 
                       [jsinterp.c, line 663]
     
  js_Interpret() 
                       [jsinterp.c, line 2290]

Comment 14

[sairuh (rarely reading bugmail)]

Attachment: better macsbug log using 2000021112

Comment 15

[Stephen P. Morse]

A fix was checked in to wallet.cpp last night that might account for what 
seeing.  If you could test this out with today's build it would be greatly 
appreciated.  Thanks.

Comment 16

[sairuh (rarely reading bugmail)]

tested using today's mac comm bits, 2000-02-14-13, and seamonkey didn't hang
when i followed the original recipe for Safe Form Fill. marking as wfm.

Comment 17

[sairuh (rarely reading bugmail)]

verif.

Comment 18

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/0fa8b12487a7
[fenix] For https://github.com/mozilla-mobile/fenix/issues/26335: Enable sponsored shortcuts by default
https://hg.mozilla.org/releases/mozilla-beta/rev/05920d14d4a5
[fenix] For https://github.com/mozilla-mobile/fenix/issues/26335: Allow for more inflations in performance tests
https://hg.mozilla.org/releases/mozilla-beta/rev/f4af2a280abf
[fenix] For https://github.com/mozilla-mobile/fenix/issues/26335: Disable tests for the number of inflations.