39723 - two request (second bad) sent for click on image map

Status: VERIFIED FIXED

(Core :: Networking, defect, P3)

Description

[bart.van.riel]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win95; en-US; m16) Gecko/20000517
BuildID:    2000051720

go to the specified URL. Click on one of the links in the picture with the cars,
e.g. "FILE" (which means 'traffic jam', it' a dutch site). Apparently the
browser does not recognize what is offered him, because it doen't show the right
page. Instead, it shows what appears to be a directory structure, like on an FTP
site.
Try it with NC4.7x or MSIE 4/5 to see the correct functionality.

Reproducible: Always
Steps to Reproduce:
1.go to the specified URL.
2.Click on one of the links in the picture with the cars, e.g. "FILE" (which
means 'traffic jam', it' a dutch site).
3.

Actual Results:  Apparently the browser does not recognize what is offered him,
because it doen't show the right page. Instead, it shows what appears to be a
directory structure, like on an FTP site.

Expected Results:  It should have displayed a web page.

Comment 1

[Doron Rosenberg (IBM)]

<PRE>
<A href="http://verkeer.anwb.org/hp_verkeersinformatie-20000404.mmp"><img border
=0 src="http://verkeer.anwb.org/anwbbin/hp_verkeersinformatie-20000404.gif"
ISMAP alt="Homepage Verkeersinformatie"></a></td></tr></PRE>

This causes the bug, ISMAP ahref'ed to a .mmp file. Confirming, not sure where
to move this.

Comment 2

[ekrock's old account (dead)]

nsbeta2 6/1-. html 3.2 b.c.

Comment 3

[Marc Attinasi]

This is an image map (ISMAP) but it behaves the same as just hitting the URL 
directly:
  http://verkeer.anwb.org/hp_verkeersinformatie-20000404.mmp

I do not knwo what mmp files are, or how they interact with image maps, however 
it looks pretty much like an image map related issue.

Assigning to pollman and attaching a reduced testcase.

Comment 4

[Marc Attinasi]

Attachment: Reduced testcase showing issue with ISMAP on an mmp file.

Comment 5

[leger]

Putting on [nsbeta2+][6/15] radar.

Comment 6

[Eric Pollmann]

We must not be sending the coordinates of the click to the server side image 
map, i.e.:

  http://verkeer.anwb.org/hp_verkeersinformatie-20000404.mmp?80,80

Comment 7

[Eric Pollmann]

We are submitting the coordinates of the image map correctly to the server.  The 
server then responds with:

HTTP/1.0 302 Found
Date: Thu, 08 Jun 2000 05:41:53 GMT
Server: Apache/1.1.3
Location: http://verkeer.anwb.org/tvn/
Content-type: text/html
...

We then request the image a second time at the correct location.  This time, we 
do not send the coordinates.

Sounds like a necko issue to me, CC'ing some folks who might know.

Comment 8

[Eric Pollmann]

Actually, this may not be a networking thing - we write the second, flawed 
request before getting the response back for the first one.  I'm wondering if 
both the anchor <a 
href="http://verkeer.anwb.org/hp_verkeersinformatie-20000404.mmp"> and the image 
map are firing OnLinkClick events, which may be causing this double request...

Comment 9

[Eric Pollmann]

Arg, yup - that's it.  The link click event is generated in both 
nsImageFrame::TriggerLink then later in nsGenericElement::TriggerLink.  Removing 
the network folks to prevent further spammage.  Sorry!  :)

Comment 10

[Eric Pollmann]

I have a fix in my tree, it's a bit of a hack and needs a better solution.  My 
hack was to have nsHTMLImageElement not delegate click events to the generic 
element and leave it to the frame, since the frame handles all sorts of image 
map cases.

The problem is I need to figure out how to differentiate and handle the simple 
case of <a><img></a> (not a map), because currently with my fix, clicking on an 
image like this doesn't do anything...

Comment 11

[Eric Pollmann]

Okay, I have a fix that works for this and the basic image-inside-anchor case.  
I'd love to go through an suite of image maps to be sure I'm not breaking 
anything though...

Comment 12

[Eric Pollmann]

Just checked in the fix.  To verify, go to:

http://verkeer.anwb.org

Click on the Flashing word "FILE"

If you see a map, the bug is fixed.  Otherwise, you will see a page of text.

Thanks!

Comment 13

[Chris Petersen]

Fixed in the June 19th build (2000061908).