18501 - submission problems with http://maps.mcom.com

Status: VERIFIED FIXED

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

Description

[Scott MacGregor]

I don't know where this bug lies so I'm going to start with html form
submissions since that was the last action I made before I saw the problem.


I wanted to look up a conference room on our internal maps server.
I went to http://maps and typed in the name of the conference room
in the html text edit box.


I then hit enter and nothing really happened. Well something did, the form
did get cleared from the window but the page with the map information
never actually loaded next. So I never got the new page with the
map search results that showed me where the conference room was.

This was using the M11 11/9 release build on windows.

Comment 1

[karnaze (gone)]

Eric, can you take a look at this and reassign as appropriate.

Comment 2

[Eric Pollmann]

Looks like Nav calls the onchange handler before submit when pressing Enter in a
single text field form.  I'll migrate this behaviour over

Comment 3

[Eric Pollmann]

After careful consideration, I've decided that I probably won't get this bug in
for M12.  Currently I have nearly 50 bugs scheduled for M13, so there is a
possibility that this bug may need to be moved out farther still.

Comment 4

[gerardok]

QA Contact update.

Comment 5

[ekrock's old account (dead)]

Setting M16. We should try to fix this before final as it would be bad if we
break all the single-field forms on the web.

Comment 6

[Eric Pollmann]

*** Bug 25083 has been marked as a duplicate of this bug. ***

Comment 7

[Eric Pollmann]

Rescheduling

Comment 8

[Phil Peterson]

This bug is really in my way. It's one of the few things I need to use 4.7 to 
do, so I'm nominating for dogfood. Is our internal maps page really the only web 
site relying on this onchange behavior?

Comment 9

[leger]

Setting to [nsbeta2+][dogfood-] radar.  Spoke with pollmann, not to common.

Comment 10

[Eric Pollmann]

I have a partial fix, to get the OnChange event sent for the text box before the 
frame is submitted.  This causes a locally mirrored version of the page to work, 
but there is some other problem with http://maps (I think javascript may be 
barfing because the top frame doesn't have permission to change the location of 
the bottom frame).

Comment 11

[Eric Pollmann]

Attachment: partial fix - send GfxTextControl onchange before submitting form

Comment 12

[Eric Pollmann]

Moving to M16 per nsbeta2+.

Comment 13

[Eric Pollmann]

*** Bug 35102 has been marked as a duplicate of this bug. ***

Comment 14

[Eric Pollmann]

See also Bug 34587, which may be a dup.

Comment 15

[Eric Pollmann]

CC'ing Harish for crash in nsParser

Comment 16

[Eric Pollmann]

The fix above will cause us to work for:

http://maps.mcom.com

But not work for:

http://maps

I'm guessing javascript thinks that those are two different domains and isn't 
letting one frame update the other for that reason?

Comment 17

[Eric Pollmann]

Reported the http://maps failure as bug 39386.  I'll check in the above fix as 
soon as I get it reviewed.

Comment 18

[Eric Pollmann]

Fix checked in.  To verify, go to http://maps.mcom.com and enter your login id.  
Using http://maps as a shortcut won't work, but this is already reported as bug 
39386.  Thanks!

Comment 19

[ckritzer (gone)]

Marking VERIFED FIXED on:
 - MacOS9 2000-05-23-12-M16 Commercial Build
 - Linux6 2000-05-23-10-M16 Commercial Build
 - Win95 2000-05-23-10-M16 Commercial Build

Comment 20

[Daniel Veditz [:dveditz]]

Adding keyword to bugs which already show a nsbeta2 triage value in the status 
whiteboard so the queries don't get screwed up.

Comment 21

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/eaa7e58f96b6
[fenix] Closes https://github.com/mozilla-mobile/fenix/issues/18501: Enable Nimbus on Beta channel