[kaffe] Bug report: MouseListeners, getComponentAt

Tue Feb 18 03:16:01 2003

--0-2081442182-1045567027=:67377
Content-Type: text/plain; charset=us-ascii
Content-Id: 
Content-Disposition: inline

hi David,

thanks for the bug report. I've looked into it and
created an updated version of the test (attached).

It seems to show (among other things ;), that kaffe's
AWT event handling sometimes can't decide properly
which event propagation model to use: the one from jdk
1.0 (propagate up the component tree untill someone
handles it), or the one from jdk 1.1 (events are
handled at component tree leaves). 

So if you run the updated test on  JDK 1.1 and click
on the button, you should see a 'clicked' message,
while clicking on other  leaves of the component tree
will result in no message, because jdk 1.1 uses the
1.1 event model to (not) propagate the event up to the
frame.

topic@clerks:~/bugs/novalis>
/usr/lib/j2sdk1.4.1_01/bin/java DemoPanelBug
enabled: true
DemoPanelBug[frame0,0,0,50x88,layout=java.awt.GridLayout,title=,resizable,normal]
java.awt.Canvas[canvas0,5,25,25x29]
java.awt.Label[label0,30,25,25x29,align=left,text=a]
java.awt.Button[button0,5,54,25x29,label=b]
java.awt.Panel[panel0,30,54,25x29,layout=java.awt.FlowLayout]
event:
java.awt.event.MouseEvent[MOUSE_CLICKED,(9,10),button=1,modifiers=Button1,clickCount=1]
on button0
Clicked
DemoPanelBug[frame0,0,0,50x88,layout=java.awt.GridLayout,title=,resizable,normal]
at java.awt.Point[x=9,y=10]

If you run the updated test on kaffe, you'll notice
that it propagates the event unless it's been handled
by the component, which is the jdk 1.0 model.

topic@clerks:~/bugs/novalis>
~/kaffe-sound-shared/bin/java DemoPanelBug
enabled: true
DemoPanelBug[frame0,0,0,96x124,layout=java.awt.GridLayout,resizable,title=]
java.awt.Canvas[canvas0,0,22,48x47]
java.awt.Label[null,48,22,48x47,Label: a]
java.awt.Button[null,0,69,48x47,Label: b]
java.awt.Panel[panel0,48,69,48x47,layout=java.awt.FlowLayout]
event:
java.awt.MouseEvt[MOUSE_CLICKED,(68,43),mods=16,clickCount=1]
on frame0
Clicked java.awt.Label[null,48,22,48x47,Label: a] at
java.awt.Point[x=68,y=43]
event:
java.awt.MouseEvt[MOUSE_CLICKED,(22,44),mods=16,clickCount=1]
on frame0
Clicked java.awt.Canvas[canvas0,0,22,48x47] at
java.awt.Point[x=22,y=44]
event:
java.awt.MouseEvt[MOUSE_CLICKED,(76,95),mods=16,clickCount=1]
on frame0
Clicked
java.awt.Panel[panel0,48,69,48x47,layout=java.awt.FlowLayout]
at java.awt.Point[x=76,y=95]
event:
java.awt.MouseEvt[MOUSE_CLICKED,(25,28),mods=16,clickCount=1]
on java.awt.Button
Clicked java.awt.Canvas[canvas0,0,22,48x47] at
java.awt.Point[x=25,y=28]

I'll first have to fix the toString methods of the
classes involved, though. You could try to take a look
at the event propagation in kaffe's
java.awt.Component.java, Button.java etc. and try to
find out why it's using the inappropriate event
handling model. Of course, pointers to documentation
describing when one should pick which event
propagation model are welcome, too, as I find the docs
on that from Sun somewhat lacking ;)

cheers,
dalibor topic

__________________________________________________
Do you Yahoo!?
Yahoo! Shopping - Send Flowers for Valentine's Day
http://shopping.yahoo.com
--0-2081442182-1045567027=:67377
Content-Type: application/octet-stream; name="DemoPanelBug.java"
Content-Transfer-Encoding: base64
Content-Description: DemoPanelBug.java
Content-Disposition: attachment; filename="DemoPanelBug.java"

LyoKVG8gdHJpZ2dlciB0aGUgYnVnOiBDbGljayBvbiBlYWNoIG9mIHRoZSBm
b3VyIHF1YXJ0ZXJzIG9mIHRoZSBncmlkLCBpbgpjbG9ja3dpc2Ugb3JkZXIu
CgpLYWZmZSBmcm9tIERlYmlhbi4gLXZlcnNpb24gZ2l2ZXMgCidFbmdpbmU6
IEp1c3QtaW4tdGltZSAgIFZlcnNpb246IDEuMC43ICAgSmF2YSBWZXJzaW9u
OiAxLjEnCgpPdXRwdXQgaXM6CkNsaWNrZWQgamF2YS5hd3QuQ2FudmFzW2Nh
bnZhczAsMCwwLDQ4eDYyXSBhdCBqYXZhLmF3dC5Qb2ludFt4PTMzLHk9NDBd
CkNsaWNrZWQgamF2YS5hd3QuTGFiZWxbbnVsbCw0OCwwLDQ4eDYyLExhYmVs
OiBhXSBhdCBqYXZhLmF3dC5Qb2ludFt4PTY0LHk9MzZdCkNsaWNrZWQgamF2
YS5hd3QuUGFuZWxbbnVsbCw0OCw2Miw0OHg2MixsYXlvdXQ9amF2YS5hd3Qu
Rmxvd0xheW91dF0gYXQgamF2YS5hd3QuUG9pbnRbeD02OSx5PTk4XQoKU3Vu
J3MgSlJFIDEuMS4gLXZlcnNpb24gZ2l2ZXMgJ2phdmEgdmVyc2lvbiAiMS4x
LjgiJy4gTm8gb3V0cHV0CgoKV2h5IHRoaXMgaXMgYSBzZXJpb3VzIGJ1Zzog
QmVjYXVzZSBwcm9ncmFtcyB3cml0dGVuIHdpdGggS2FmZmUgdG8KZGVwZW5k
IG9uIGdldHRpbmcgdGhvc2UgY2xpY2tzIHdpbGwgYnJlYWsgb24gdGhlIFN1
biBKUkUuICBJIGJlbGlldmUKdGhlIGJlaGF2aW9yIGlzIHRoZSBzYW1lIG9u
IGxhdGVyIEpSRXMsIGJ1dCBpdCBjb3VsZCBiZSBzb21lIG90aGVyCmJyZWFr
YWdlIGNhdXNpbmcgc2ltaWxhciBzeW1wdG9tcy4gIEkgZG9uJ3QgaGF2ZSBl
YXN5IGFjY2VzcyB0bwphbnl0aGluZyBidXQgMS4xLgoKICovCmltcG9ydCBq
YXZhLmF3dC4qOwppbXBvcnQgamF2YS5hd3QuZXZlbnQuKjsKCnB1YmxpYyBj
bGFzcyBEZW1vUGFuZWxCdWcgZXh0ZW5kcyBGcmFtZSB7CiAgICBjbGFzcyBU
ZXN0TGlzdGVuZXIgZXh0ZW5kcyBNb3VzZUFkYXB0ZXIgewoJcHVibGljIHZv
aWQgbW91c2VDbGlja2VkIChNb3VzZUV2ZW50IGUpIHsKCSAgICBTeXN0ZW0u
b3V0LnByaW50bG4oImV2ZW50OiAiICsgZSk7CgkgICAgT2JqZWN0IGNsaWNr
ZWQgPSBnZXRDb21wb25lbnRBdCAoZS5nZXRQb2ludCAoKSk7CgkgICAgU3lz
dGVtLm91dC5wcmludGxuICgiQ2xpY2tlZCAiICsgY2xpY2tlZCArICIgYXQg
IiArIGUuZ2V0UG9pbnQgKCkpOyAgIAoJfQogICAgfQoKICAgIHB1YmxpYyBE
ZW1vUGFuZWxCdWcgKCkgewoJc2V0TGF5b3V0IChuZXcgR3JpZExheW91dCAo
MiwgMikpOwoKCUNvbXBvbmVudCBwYXJ0cyBbXSA9IG5ldyBDb21wb25lbnQg
W10gewoJICAgIG5ldyBDYW52YXMgKCksIG5ldyBMYWJlbCAoImEiKSwgCgkg
ICAgbmV3IEJ1dHRvbiAoImIiKSwgbmV3IFBhbmVsICgpfTsKCglmb3IgKGlu
dCBpID0gMDsgaSA8IDQ7IGkrKykgewoJICAgIGFkZCAocGFydHMgW2ldKTsK
CX0KCgkvLyBhZGQgYSBtb3VzZSBsaXN0ZW5lciB0byB0b3AgZnJhbWUgdG8g
c2hvdwoJLy8gdGhhdCBldmVudHMgcHJvcGFnYXRlIHVwIHdoZW4gdGhleSBh
cmUgbm90CgkvLyBoYW5kbGVkCglhZGRNb3VzZUxpc3RlbmVyIChuZXcgVGVz
dExpc3RlbmVyKCkpOwoKCS8vIGFkZCBhIG1vdXNlIGxpc3RlbmVyIHRvIHRo
ZSBidXR0b24sIHRvCgkvLyBzaG93IHRoYXQgZXZlbnRzIGFyZSByZWNlaXZl
ZCBieSBsYWlkCgkvLyBvdXQgY29tcG9uZW50cy4KCXBhcnRzWzJdLmFkZE1v
dXNlTGlzdGVuZXIobmV3IFRlc3RMaXN0ZW5lcigpKTsKCglwYWNrICgpOwoJ
c2hvdyAoKTsKCglTeXN0ZW0ub3V0LnByaW50bG4oImVuYWJsZWQ6ICIgKyBp
c0VuYWJsZWQoKSk7CglTeXN0ZW0ub3V0LnByaW50bG4odGhpcyk7Cglmb3Ig
KGludCBpID0gMDsgaSA8IHBhcnRzLmxlbmd0aDsgKytpKSB7CgkgICAgU3lz
dGVtLm91dC5wcmludGxuKHBhcnRzW2ldKTsKCX0KICAgIH0KICAgIC8vdW5p
bXBvcnRhbnQgc3R1ZmYKCiAgICBwdWJsaWMgc3RhdGljIHZvaWQgbWFpbiAo
U3RyaW5nIGFyZ3MgW10pIHsKCW5ldyBEZW1vUGFuZWxCdWcgKCk7CiAgICB9
Cn0K

--0-2081442182-1045567027=:67377--