52468 - Can't receive map information from Netscape http://maps

Status: VERIFIED FIXED

(Core :: Layout: Form Controls, defect, P3)

Description

[Jaime Rodriguez, Jr.]

Build ID: 2000091108
Reproducible: Always

Steps:
1. Launch N6
2. Enter maps into the URL field
3. Receive the Netscape Campus "The Locator" page
4. Enter the name of the person you wish to find

Result: No information is returned to the bottom frame

Comment 1

[John Unruh]

Adding dogfood keyword.

Comment 2

[Asa Dotzler [:asa]]

I'm getting a "focusedWindow.location has no properties in
chrome://navigator/content/navigator.js line 119" message in the javascript
console. over to jst.

Comment 3

[Jim Roskind]

This is actually a day-to-day activity for Netscaper's, and would make folks
have to use Navv 4.x, rather than N-6.... so I'll mark this as dogfood-plus.
This would not stop a N-6 ship, but getting it working will help folks use/test
N6, which in turn will help find high visibility bugs.

Comment 4

[Doron Rosenberg (IBM)]

what component is this?

Comment 5

[Jaime Rodriguez, Jr.]

navigator

Comment 6

[Johnny Stenback (:jst)]

Reassigning to rods, turns out that the input field in the upper frame on maps
has an onchange handler that triggers the search, but in mozilla, if you type a
name in the input and hit enter, we don't generate an onchange event. If you
type a name and click anywhere outside the input field the search will be
triggerd. Rod, I'm not sure if this fix should go into the
nsGfxTextControlFrame2 code or into nsHTMLInputElement, either one could work I
suppose...

Cc:ing pollmann, heikki, joki and jay

Comment 7

[rods (gone)]

Reassigning to editor. I would have put the notification in 
nsHTMLInputElement.cpp, and there is a really easy spot to it but.....

When you hit return the text input appears to remember the state of the content 
and then the next time you hit return it compares the current state with that 
state. If the state has changed in between the two times you have hit return, 
then and only then is it sent.

In Nav 4.x you can type in "ABC" and hit return and the onchange will fire. Then 
you can type in DEF (so the string is now ABCDEF), now without hitting return 
backspace three times and then hit return. The onchange is not fired. 

Summary:
The OnChange is fired only if the contents of the input text has changed since 
the last time you hit return (or if it is the first time you hit return).

This is an editor thing because they will have to have someway to remember the 
state each time the return key is hit. One option is to change 
nsHTMLInputElement.cpp to generate the OnChange, but in that piece of code you 
would have to ask the input content if it had changed since the last time return 
was hit.

Comment 8

[Jay Carson]

By the way, I'm working on fixing the animation part of The Locator
to now support Nav6, so hopefully we are close to getting The Locator to being
fully functional, in every way, with Nav6.  And I really appreciate everybody's
efforts here, as well as in past Locator-spawned Nav6 mozilla bug reports.

I figured out how to do the animation in Nav6:

     http://ledzep.red.iplanet.com/u/jay/xfer/dom6.shtml

so now I just need to migrate this code into The Locator's final DHTML.

(I'm the author/creator of The Locator, by the way  :>  )

Thanks everybody!

Comment 9

[Doron Rosenberg (IBM)]

setting default qa

Comment 10

[kinmoz]

Assigning to [e-mail], he has a fix for this already.

Comment 11

[kinmoz]

=

Comment 12

[Jim Roskind]

If you have a fix in hand... can we get an attached patch, and start the review
process?

Comment 13

[mjudge]

fixed.

Comment 14

[mjudge]

fixed. sorry no resoultion

Comment 15

[bsharma]

Updating QA contact.

Comment 16

[bsharma]

Verified on 2000092711 build.
Platform: WinNT

When a person name is entered, the complete information is displayed in the
bottom screen.

Comment 17

[Matthias Versen [:Matti]]

mid-air collision ? / bugzilla cleanup
Reopening (current State: verified and no resolution

Comment 18

[Matthias Versen [:Matti]]

really reopening
* SORRY for the SPAM *

Comment 19

[Matthias Versen [:Matti]]

fixed

Comment 20

[Matthias Versen [:Matti]]

verfied