3553 - [BLOCK] Linux: Links do not work.

Status: VERIFIED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P1)

Description

[Chris McAfee]

Linux, viewer & apprunner.  Links don't work, but instead
print out

  debug reset selection has been called
  debug reset selection has been called

Comment 1

[Akkana Peck]

"debug reset selection has been called" is a message from the selection code
(nsRangeList.cpp),  and I thought it was annoying, too, and commented it out;
whoever cared about it can uncomment it in his own tree.  But the
links-not-working problem isn't related to that debug message; something must
have changed recently in the event handling, maybe mouse events over links
aren't being passed to the right handler any more.

Comment 2

[troy]

Tom, since it's event handling I'm changing the component and assigning it to
you

Comment 3

[Chris McAfee]

Marking important.

Comment 4

[Peter Trudelle]

setting (desired) target M3

Comment 5

[joki (gone)]

Looked into this with akkana this afternoon.  I have some clues but still
trying to figure out the exact cause.

Comment 6

[leger]

janc, can you check if this is Linux specific...I think so.  But can you check
Mar10 Win and Mac please?

Comment 7

[Jan Carpenter]

3/10/99 build
works fine on Win98, WinNT, and Mac PPC.

Comment 8

[Syd Logan]

It looks to me like the x and y point for the event being computed in
nsViewManager() has wrong values, this could lead to nothing hitting when we
search for a link to follow. This could be because we don't have a native
GetBounds() in nsWindow.cpp; the values we are getting looked wrong to me. I
code a native GetBounds() and see if that fixes things.

Comment 9

[Chris McAfee]

Links turn green when I click on them, this implies that
we've got the position right.

Comment 10

[Syd Logan]

Hmmm, they don't for me, however. No click-on-me hand icon either... (I just
pulled sources an hour ago).

Comment 11

[Akkana Peck]

With sources pulled this morning, links turn red when clicked (i.e., mouse down
still works) but mouse up doesn't load the referenced page.  It seems like the
mouse down event works, but the mouse up event fails.

Joki and I looked at this very briefly a few days ago and it was seeing the
mouse up event as happening on a different widget from the mouse down event.
Oddly, it seemed to be saying that the mouse down (which worked) was over an
image whereas the mouse up (which didn't work) was over a link, which seems
backward.

Comment 12

[Chris McAfee]

Linux/Solaris: links turn green in apprunner, red in viewer.
AIX: syd isn't seeing any color, sounds like there's a position problem
     that's compounding this problem.

Comment 13

[Syd Logan]

The huge x, y coordinates I was seeing are in twips, so not a problem...

Comment 14

[Chris McAfee]

joki is out until Saturday.

The problem seems to have something to do with the mouse-down frame
not matching the mouse-up frame, thus layout thinks this is not
a click but a ? drag.  Since we're failing this test, we don't
get the link click.

Comment 15

[joki (gone)]

Fixed.  Reentrant focus event on Linux was interrupting mousedown processing.
Working on long term issues but marking this fixed.

Comment 16

[joki (gone)]

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

Comment 17

[leger]

jan c, please verify with Mar16 build when out.  Thanks!

Comment 18

[Claudius Gayle]

VERIFIED-FIXED for linux apprunner (Mar16) although they are still slow as molasses...