318678 - [FIX]events registered via divElem.onmouseover and divElem.onclick do not trigger

Status: RESOLVED FIXED

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

Description

[Daniel Ceregatti]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.8) Gecko/20051201 Firefox/1.5
Build Identifier: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.8) Gecko/20051201 Firefox/1.5

Go to http://map.sh.nu/prototype4/ and allow popups for the site. When the site loads, a "debug" window which is created entirely in DOM will pop up. In this window there is the text "scalebarImg" and "tiles" at the top left. These are divs with text nodes appended. The divs are created from the source here:

http://map.sh.nu/w/prototype4/debug.txt

The 4 events are registered on lines 122, 125, 141, and 144. In firefox 1.0.7 this works perfectly. In firefox 1.5 it does not. If I use venkman to "slow down" the execution of the script by adding breakpoints the above lines, the onclick events begin to work, but the onmouseover still does not.

The javascript console doesn't report anything when I mouse over or click on the divs elements.

Even though I'm on Linux X86_64, I've confirmed this on Linux x86 and windows x86.

Reproducible: Always

Comment 1

[Johnny Stenback (:jst)]

Taking bug, but bz, feel free to have a look if you've got some cycles.

Comment 2

[Andrew Schultz]

this regressed between 2005-07-30-05 and 2005-07-31-10 (landing of 296639)

Comment 3

[Andrew Schultz]

(regrression was for onclick specifically)

Comment 4

[Boris Zbarsky [:bzbarsky]]

So I added some logging to nsEventListenerManager::RegisterScriptEventListener to see what's going on as follows:

  fprintf(stderr, "Registering listener for %s on %p\n", NS_ConvertUTF16toUTF8(type).get(), jsobj);

Here's what I see for the div in question (by matching up pointer values):

Created wrapped native [object HTMLDivElement @ 0xb2b67d18 (native @ 0xb2b67cf0)], flat JSObject is 0xb2b2ca88
Registering listener for onclick on 0xb2b2ca88
Registering listener for onclick on 0xb2b2ca88

and then much later (probably as I click around):

Created wrapped native [object HTMLDivElement @ 0xb2b9a340 (native @ 0xb2b67cf0)], flat JSObject is 0xb2b6f450
Created wrapped native [object HTMLDivElement @ 0xb2b9baf0 (native @ 0xb2b67cf0)], flat JSObject is 0xb3d5b6c0
Created wrapped native [object HTMLDivElement @ 0xb3c16f20 (native @ 0xb2b67cf0)], flat JSObject is 0xb4dfa950

So for some reason we "lose" the original XPCWrappedNative that the event listener is set on... hence no listener firing.

This is all for the onclick listener; the onmouseover listener is a bug in the site (but Daniel, please don't change the code until we can get to the bottom of this, ok?).

Comment 5

[Boris Zbarsky [:bzbarsky]]

Attachment: Fix

Fix -- have to use the correct old scope when reparenting wrappers...

Comment 6

[Boris Zbarsky [:bzbarsky]]

I think we want this on the 1.8 branch... I sorta wish I'd followed up more on my questions about this code in bug 296639.  :(

Comment 7

[Johnny Stenback (:jst)]

Comment on attachment 204852 [details] [diff] [review]
Fix

r+sr=jst

Comment 8

[Boris Zbarsky [:bzbarsky]]

Checked in to trunk.

Comment 9

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 204852 [details] [diff] [review]
Fix

I think we need this on the 1.8 branch as soon as we can manage... :(

Comment 10

[Mike Schroepfer]

Comment on attachment 204852 [details] [diff] [review]
Fix

Please land for 1.8.0 and 1.8 branches.  Thanks!

Comment 11

[Boris Zbarsky [:bzbarsky]]

Could someone please land this for me?  If not, I'll do it in early January...

Comment 12

[Boris Zbarsky [:bzbarsky]]

Attachment: Patch merged to 1.8 branch

Comment 13

[Boris Zbarsky [:bzbarsky]]

*** Committing to MOZILLA_1_8_BRANCH... 
Checking in content/base/public/nsContentUtils.h;
new revision: 1.60.4.5; previous revision: 1.60.4.4
Checking in content/base/src/nsContentUtils.cpp;
new revision: 1.107.4.9; previous revision: 1.107.4.8

*** Committing content/base/public/nsContentUtils.h on MOZILLA_1_8_0_BRANCH... 
new revision: 1.60.4.4.2.1; previous revision: 1.60.4.4
*** Committing content/base/src/nsContentUtils.cpp on MOZILLA_1_8_0_BRANCH... 
new revision: 1.107.4.8.2.1; previous revision: 1.107.4.8