6399 - form submission broken in viewer

Status: VERIFIED FIXED

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

Description

[Ramiro Estrugo]

Try a form submission on the above url with the viewer on either windows or
linux.

The viewer will hang.

IT works on the apprunner.

There are some messages to the effect of connecting to people.netscape.com.
I have no idea why a simple form submission would fetch urls from
people.netscape.com.

Comment 1

[Stephen P. Morse]

The connection to people.netscape.com is not the problem -- those connections
are established properly.  But since it was the last successful action
performed, it is the last thing visible on the console screen at the time of the
hang.

The problem is in using the service manager to access kFileLocatorCID.  This
occurs at the very beginning of the Wallet_ProfileDirectory routine in
mozilla/extensions/wallet/src/wallet.cpp.  Because of this failure, wallet is
unable to open the user's database and is asking if it should retry.  But
because we don't yet have dialogs, the dialog that asks this question is
invisible and automatically returns a value of YES.  Hence we keep retrying
forever.

So the issue is a viewer issue because viewer probably doesn't contain the dll
for FileLocator.  But are we really concerned about viewer anymore?  I thought
that was going away.  So I'm going to mark this as wont-fix unless I hear back
that we do still need to support viewer.

One final point -- you asked why a simple form submission is fetching urls from
people.netscape.com.  That's because when you submitted the form you said that
you wanted wallet to capture the data that was on the form.  But of course you
don't remember ever saying that because it was asked to you on a dialog window
and, since dialogs are not yet implemented, you answer was taken as a yes by
default.  If you want to really see these dialogs, you can set the preference of
wallet.useDialogs in your prefs file (you have to do this manually) and the
dialogs will appear in the console window.  Then you can say "no" to the
question of capturing the data and you won't fetch those urls nor will you see
this hang.

Comment 2

[cpratt]

Viewer should be going away.

Comment 3

[Chris McAfee]

I don't think viewer is going away anytime soon,
this is a call the layout people need to make.

Comment 4

[rickg]

Chris -- if this is a general forms problem it should be fixed, because while
we're not shipping viewer, it is our testbed. If it's wallet related, or
non-essential then mark it as won't fix.

Comment 5

[karnaze (gone)]

Steve, you should fix this bug even before the new changes to wallet that we
discussed go into effect. Forever calling a dialog seems like something that can
easily be fixed.

Comment 6

[Stephen P. Morse]

OK, I put in a fix so that the dialog for trying again now returns "no" if there
are no dialogs.  That stops the hang.

Comment 7

[cpratt]

I can't actually regress this as there is no viewer in current seamonkey builds
on NT - so I'm going to go ahead and mark it as verified. ramiro, if there's
still a problem, please re-open. Thanks!