7583 2006-03-03 17:18:59 -0800 DRT hangs when doing eventSender.mouseDown on native widgets 2006-03-09 08:36:16 -0800 1 1 1 Unclassified WebKit HTML Editing 420+ Mac OS X 10.4 RESOLVED FIXED P2 Normal --- 1 justin.garcia webkit-unassigned mitz oldest_to_newest 35022 0 justin.garcia 2006-03-03 17:18:59 -0800 Attaching a test that hangs 35023 1 6835 justin.garcia 2006-03-03 17:19:39 -0800 Created attachment 6835 test that hangs 35024 2 justin.garcia 2006-03-03 17:20:35 -0800 <rdar://problem/4454360> layout tests that use eventSender hang 35032 3 6838 mitz 2006-03-04 01:46:10 -0800 Created attachment 6838 Reduction The problem here is that when you mousedown in a native widget (in this case a text field), it enters its own mouse-tracking runloop until the mouse is released. (The reason this doesn't happen with the first mousedown in the first attachment is that the onfocus handler fires first; you will probably want to call test() only once, either as onfocus handler or from step1, and also add a mouseup between the two mousedowns, if you can get around this bug). 35314 4 justin.garcia 2006-03-06 11:46:29 -0800 Vicki handed this testcase off to me, and I hadn't even looked at it when I brought it over to radar. She wanted to create a testcase for the fix for 7363. The fact that there is a mouseDown with no mouseUp just looks like a mistake, and not something that is necessary to test her fix. I'm closing this bug. 35315 5 mitz 2006-03-06 11:56:04 -0800 (In reply to comment #4) In case it wasn't clear from my previous comment, the missing mouseUp is not what's making the test hang. I think it might be possible to test the fix without mouse events by calling focus() on the text fields. 35323 6 justin.garcia 2006-03-06 13:17:13 -0800 (In reply to comment #3) > The problem here is that when you mousedown in a native widget (in this case a > text field), it enters its own mouse-tracking runloop until the mouse is > released. I thought that this meant that we would just need to send a mouseUp event. Sending a mouseUp has no effect. I'm reopening. > I think it might be possible to test the fix without mouse events by calling > focus() on the text fields. I agree, I wrote two testcases for Vicki's fix for 7363 and I'll check them in once she reviews them. 35487 7 justin.garcia 2006-03-07 22:01:12 -0800 This is b/c JS is suspended while the mouse is down over native widgets. Mitz's idea is to add a click method to eventSender. I'm going to let him work on this. 35541 8 6943 mitz 2006-03-08 09:41:25 -0800 Created attachment 6943 Add eventSender.click() for native widgets This implementation of click should only be used with native widgets. Implementing a second "convenience" code path that simply does a mouseDown followed by a mouseUp requires [WebHTMLView _hitViewForEvent:], which is internal. 35625 9 6943 darin 2006-03-09 07:42:36 -0800 Comment on attachment 6943 Add eventSender.click() for native widgets OK, r=me 6835 2006-03-03 17:19:39 -0800 2006-03-03 17:19:39 -0800 test that hangs no-renderer.html text/plain 1232 justin.garcia PGh0bWw+CjxoZWFkPgo8c2NyaXB0IHR5cGU9InRleHQvamF2YXNjcmlwdCI+CnZhciBzdGFydFgg PSAyMDsKdmFyIHN0YXJ0WSA9IDIwOwoKZnVuY3Rpb24gbG9nKG1lc3NhZ2UpIHsKICAgIHZhciBj b25zb2xlID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImNvbnNvbGUiKTsKICAgIHZhciBsaSA9 IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoImxpIik7CiAgICB2YXIgdGV4dCA9IGRvY3VtZW50LmNy ZWF0ZVRleHROb2RlKG1lc3NhZ2UpOwogICAgCiAgICBjb25zb2xlLmFwcGVuZENoaWxkKGxpKTsK ICAgIGxpLmFwcGVuZENoaWxkKHRleHQpOwp9CgpmdW5jdGlvbiB0ZXN0KCkKewoJYi5zdHlsZS5k aXNwbGF5ID0gJ25vbmUnOwoJYS52YWx1ZSA9ICdub3cgY2xpY2sgaGVyZSc7Cn0KCmZ1bmN0aW9u IHJ1blRlc3QoKQp7CiAgICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKSB7CiAgICAg ICAgbGF5b3V0VGVzdENvbnRyb2xsZXIud2FpdFVudGlsRG9uZSgpOwogICAgfQogICAgZWxzZSB7 CgkJcmV0dXJuOwogICAgfQoKCXdpbmRvdy5zZXRUaW1lb3V0KCdzdGVwMSgpJywgNTAwKTsKbG9n KCJydW5UZXN0Iik7Cn0KCmZ1bmN0aW9uIHN0ZXAxKCkKewpsb2cgKCJzdGVwMSIpOwoJZXZlbnRT ZW5kZXIubW91c2VNb3ZlVG8oc3RhcnRYLCBzdGFydFkpOwoJZXZlbnRTZW5kZXIubW91c2VEb3du KCk7CmxvZyAoImJlZ2luIHRlc3QiKTsKCXRlc3QoKTsKbG9nICgiZW5kIHRlc3QiKTsKCS8vIGlm IEkgY29tbWVudCBvdXQgdGhpcyBzZWNvbmQgbW91c2VEb3duLCB0aGUgaGFuZyBnb2VzIGF3YXkK CWV2ZW50U2VuZGVyLm1vdXNlRG93bigpOwoJd2luZG93LnNldFRpbWVvdXQoJ2ZpbmlzaCgpJywg NTAwKTsKfQoKCmZ1bmN0aW9uIGZpbmlzaCgpCnsKbG9nICgiZmluaXNoIik7CiAgICBpZiAod2lu ZG93LmxheW91dFRlc3RDb250cm9sbGVyKQogICAgICAgIGxheW91dFRlc3RDb250cm9sbGVyLm5v dGlmeURvbmUoKTsKfQoKCjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5Pgo8aW5wdXQgaWQ9ImIiIHR5 cGU9InRleHQiIHZhbHVlPSJjbGljayBoZXJlIiBvbmZvY3VzPSJ0ZXN0KCkiPgo8aW5wdXQgaWQ9 ImEiIHR5cGU9InRleHQiPgo8dWwgc3R5bGU9Imxpc3Qtc3R5bGU6Y2lyY2xlOyBtYXJnaW46IDFl bTsiIGlkPSJjb25zb2xlIj48L3VsPgoKPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgpy dW5UZXN0KCk7Cjwvc2NyaXB0Pgo8L2JvZHk+CjwvaHRtbD4= 6838 2006-03-04 01:46:10 -0800 2006-03-04 01:46:10 -0800 Reduction 7583 no-renderer.html text/html 306 mitz PGh0bWw+CjxoZWFkPgo8c2NyaXB0IHR5cGU9InRleHQvamF2YXNjcmlwdCI+CnZhciBzdGFydFgg PSAyMDsKdmFyIHN0YXJ0WSA9IDIwOwoKZnVuY3Rpb24gdGVzdCgpCnsKICAgIGlmICghd2luZG93 LmxheW91dFRlc3RDb250cm9sbGVyKQogICAgICAgIHJldHVybjsKICAgIGV2ZW50U2VuZGVyLm1v dXNlTW92ZVRvKHN0YXJ0WCwgc3RhcnRZKTsKICAgIGV2ZW50U2VuZGVyLm1vdXNlRG93bigpOwp9 Cjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5IG9ubG9hZD0idGVzdCgpIj4KPGlucHV0IHR5cGU9InRl eHQiPgo8L2JvZHk+CjwvaHRtbD4K 6943 2006-03-08 09:41:25 -0800 2006-03-09 07:42:36 -0800 Add eventSender.click() for native widgets 7583_r1.patch text/plain 5504 mitz SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDEzMjE0KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDYtIyMtIyMgIE1pdHogUGV0dGVsICA8b3Bl bmRhcndpbi5vcmdAbWl0enBldHRlbC5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgLSB0ZXN0IGZvciBodHRwOi8vYnVnemlsbGEub3BlbmRhcndp bi5vcmcvc2hvd19idWcuY2dpP2lkPTc1ODMKKyAgICAgICAgICBEUlQgaGFuZ3Mgd2hlbiBkb2lu ZyBldmVudFNlbmRlci5tb3VzZURvd24gb24gbmF0aXZlIHdpZGdldHMKKworICAgICAgICAqIGZh c3QvZXZlbnRzL2V2ZW50LXNlbmRlci1tb3VzZS1jbGljay1leHBlY3RlZC50eHQ6IEFkZGVkLgor ICAgICAgICAqIGZhc3QvZXZlbnRzL2V2ZW50LXNlbmRlci1tb3VzZS1jbGljay5odG1sOiBBZGRl ZC4KKwogMjAwNi0wMy0wNyAgSnVzdGluIEdhcmNpYSAgPGp1c3Rpbi5nYXJjaWFAYXBwbGUuY29t PgogCiAgICAgICAgIFJldmlld2VkIGJ5IGhhcnJpc29uCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0 L2V2ZW50cy9ldmVudC1zZW5kZXItbW91c2UtY2xpY2stZXhwZWN0ZWQudHh0Cj09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K LS0tIExheW91dFRlc3RzL2Zhc3QvZXZlbnRzL2V2ZW50LXNlbmRlci1tb3VzZS1jbGljay1leHBl Y3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2V2ZW50cy9ldmVudC1z ZW5kZXItbW91c2UtY2xpY2stZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDYg QEAKKworVGVzdCBmb3IgaHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNn aT9pZD03NTgzIERSVCBoYW5ncyB3aGVuIGRvaW5nIGV2ZW50U2VuZGVyLm1vdXNlRG93biBvbiBu YXRpdmUgd2lkZ2V0cy4gVGhpcyB0ZXN0cyBldmVudFNlbmRlci5tb3VzZUNsaWNrKCkuCisKK1Rl c3QgcmVzdWx0OiBQQVNTLgorCisKSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvZXZlbnRzL2V2ZW50 LXNlbmRlci1tb3VzZS1jbGljay5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3Qv ZXZlbnRzL2V2ZW50LXNlbmRlci1tb3VzZS1jbGljay5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5 b3V0VGVzdHMvZmFzdC9ldmVudHMvZXZlbnQtc2VuZGVyLW1vdXNlLWNsaWNrLmh0bWwJKHJldmlz aW9uIDApCkBAIC0wLDAgKzEsMjAgQEAKKzxzY3JpcHQ+Cit2YXIgY2xpY2tlZCA9IGZhbHNlOwor PC9zY3JpcHQ+Cis8aW5wdXQgdHlwZT0idGV4dCIgb25mb2N1cz0iY2xpY2tlZCA9IHRydWU7Ij4K KzxwPgorICAgIFRlc3QgZm9yIDxpPjxhIGhyZWY9Imh0dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2lu Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzU4MyI+aHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3Jn L3Nob3dfYnVnLmNnaT9pZD03NTgzPC9hPgorICAgIERSVCBoYW5ncyB3aGVuIGRvaW5nIGV2ZW50 U2VuZGVyLm1vdXNlRG93biBvbiBuYXRpdmUgd2lkZ2V0czwvaT4uIFRoaXMgdGVzdHMgZXZlbnRT ZW5kZXIubW91c2VDbGljaygpLgorPC9wPgorPGhyPgorPHA+CisgICAgVGVzdCByZXN1bHQ6IDxz cGFuIGlkPSJyZXN1bHQiPihjYW5ub3QgcnVuIGludGVyYWN0aXZlbHkpPC9zcGFuPi4KKzwvcD4K KzxzY3JpcHQ+CitpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKSB7CisgICAgbGF5b3V0 VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCgpOworICAgIGV2ZW50U2VuZGVyLm1vdXNlTW92ZVRv KDIwLCAyMCk7CisgICAgZXZlbnRTZW5kZXIubW91c2VDbGljaygpOworICAgIGRvY3VtZW50Lmdl dEVsZW1lbnRCeUlkKCdyZXN1bHQnKS5pbm5lclRleHQgPSBjbGlja2VkID8gJ1BBU1MnIDogJ0ZB SUwnOworfQorPC9zY3JpcHQ+CkluZGV4OiBXZWJLaXRUb29scy9DaGFuZ2VMb2cKPT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PQotLS0gV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCShyZXZpc2lvbiAxMzIxNCkKKysrIFdlYktpdFRv b2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE4IEBACisyMDA2LSMjLSMj ICBNaXR6IFBldHRlbCAgPG9wZW5kYXJ3aW4ub3JnQG1pdHpwZXR0ZWwuY29tPgorCisgICAgICAg IFRlc3Q6IGZhc3QvZXZlbnRzL2V2ZW50LXNlbmRlci1tb3VzZS1jbGljay5odG1sCisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgLSBmaXggaHR0cDovL2J1 Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD03NTgzCisgICAgICAgICAgRFJU IGhhbmdzIHdoZW4gZG9pbmcgZXZlbnRTZW5kZXIubW91c2VEb3duIG9uIG5hdGl2ZSB3aWRnZXRz CisKKyAgICAgICAgKiBEdW1wUmVuZGVyVHJlZS9EdW1wUmVuZGVyVHJlZS5tOgorICAgICAgICAo K1tFdmVudFNlbmRpbmdDb250cm9sbGVyIGlzU2VsZWN0b3JFeGNsdWRlZEZyb21XZWJTY3JpcHQ6 XSk6CisgICAgICAgICgtW0V2ZW50U2VuZGluZ0NvbnRyb2xsZXIgbW91c2VDbGlja10pOiBTaW11 bGF0ZXMgYSBjbGljayBpbiBhIG5hdGl2ZQorICAgICAgICB3aWRnZXQgYnkgcXVldWVpbmcgYSBt b3VzZVVwIGJlZm9yZSBzZW5kaW5nIHRoZSBtb3VzZURvd24sIHNvIHRoYXQKKyAgICAgICAgdGhl IHdpZGdldCdzIG1vdXNlIHRyYWNraW5nIGV2ZW50IGxvb3AgZG9lc24ndCBoYW5nIGluZGVmaW5p dGVseS4KKwogMjAwNi0wMy0wOCAgTWFjaWVqIFN0YWNob3dpYWsgIDxtanNAYXBwbGUuY29tPgog CiAgICAgICAgIFJldmlld2VkIGJ5IEVyaWMuCkluZGV4OiBXZWJLaXRUb29scy9EdW1wUmVuZGVy VHJlZS9EdW1wUmVuZGVyVHJlZS5tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdFRvb2xzL0R1bXBSZW5k ZXJUcmVlL0R1bXBSZW5kZXJUcmVlLm0JKHJldmlzaW9uIDEzMjE0KQorKysgV2ViS2l0VG9vbHMv RHVtcFJlbmRlclRyZWUvRHVtcFJlbmRlclRyZWUubQkod29ya2luZyBjb3B5KQpAQCAtNjg5LDYg KzY4OSw3IEBAIHN0YXRpYyB2b2lkIGR1bXAodm9pZCkKIHsKICAgICBpZiAoYVNlbGVjdG9yID09 IEBzZWxlY3Rvcihtb3VzZURvd24pCiAgICAgICAgICAgICB8fCBhU2VsZWN0b3IgPT0gQHNlbGVj dG9yKG1vdXNlVXApCisgICAgICAgICAgICB8fCBhU2VsZWN0b3IgPT0gQHNlbGVjdG9yKG1vdXNl Q2xpY2spCiAgICAgICAgICAgICB8fCBhU2VsZWN0b3IgPT0gQHNlbGVjdG9yKG1vdXNlTW92ZVRv WDpZOikKICAgICAgICAgICAgIHx8IGFTZWxlY3RvciA9PSBAc2VsZWN0b3IobGVhcEZvcndhcmQ6 KSkKICAgICAgICAgcmV0dXJuIE5POwpAQCAtODA0LDYgKzgwNSw0MCBAQCBzdGF0aWMgdm9pZCBk dW1wKHZvaWQpCiAgICAgfQogfQogCistICh2b2lkKW1vdXNlQ2xpY2sKK3sKKyAgICBbW1tmcmFt ZSBmcmFtZVZpZXddIGRvY3VtZW50Vmlld10gbGF5b3V0XTsKKyAgICBpZiAoW3NlbGYgY3VycmVu dEV2ZW50VGltZV0gLSBsYXN0Q2xpY2sgPj0gMSkKKyAgICAgICAgY2xpY2tDb3VudCA9IDE7Cisg ICAgZWxzZQorICAgICAgICBjbGlja0NvdW50Kys7CisgICAgTlNFdmVudCAqbW91c2VEb3duRXZl bnQgPSBbTlNFdmVudCBtb3VzZUV2ZW50V2l0aFR5cGU6TlNMZWZ0TW91c2VEb3duIAorICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGxvY2F0aW9uOmxhc3RNb3VzZVBvc2l0 aW9uIAorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBtb2RpZmllckZsYWdzOm5p bCAKKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIHRpbWVzdGFtcDpbc2Vs ZiBjdXJyZW50RXZlbnRUaW1lXQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg d2luZG93TnVtYmVyOltbW2ZyYW1lIHdlYlZpZXddIHdpbmRvd10gd2luZG93TnVtYmVyXSAKKyAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgY29udGV4dDpbTlNHcmFwaGlj c0NvbnRleHQgY3VycmVudENvbnRleHRdIAorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgIGV2ZW50TnVtYmVyOisrZXZlbnROdW1iZXIgCisgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIGNsaWNrQ291bnQ6Y2xpY2tDb3VudCAKKyAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICBwcmVzc3VyZTpuaWxdOworCisgICAgTlNWaWV3ICpzdWJW aWV3ID0gW1tmcmFtZSB3ZWJWaWV3XSBoaXRUZXN0Olttb3VzZURvd25FdmVudCBsb2NhdGlvbklu V2luZG93XV07CisgICAgaWYgKHN1YlZpZXcpIHsKKyAgICAgICAgW3NlbGYgbGVhcEZvcndhcmQ6 MV07CisgICAgICAgIE5TRXZlbnQgKm1vdXNlVXBFdmVudCA9IFtOU0V2ZW50IG1vdXNlRXZlbnRX aXRoVHlwZTpOU0xlZnRNb3VzZVVwCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICBsb2NhdGlvbjpsYXN0TW91c2VQb3NpdGlvbgorICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIG1vZGlmaWVyRmxhZ3M6bmlsCisgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIHRpbWVzdGFtcDpb c2VsZiBjdXJyZW50RXZlbnRUaW1lXQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICB3aW5kb3dOdW1iZXI6W1tbZnJhbWUgd2ViVmlld10gd2luZG93XSB3aW5k b3dOdW1iZXJdCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgY29udGV4dDpbTlNHcmFwaGljc0NvbnRleHQgY3VycmVudENvbnRleHRdCisgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBldmVudE51bWJlcjorK2V2 ZW50TnVtYmVyCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgY2xpY2tDb3VudDpjbGlja0NvdW50CisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICBwcmVzc3VyZTpuaWxdOworICAgICAgICBbTlNBcHAgcG9zdEV2 ZW50Om1vdXNlVXBFdmVudCBhdFN0YXJ0Ok5PXTsKKyAgICAgICAgW3N1YlZpZXcgbW91c2VEb3du Om1vdXNlRG93bkV2ZW50XTsKKyAgICB9Cit9CisKIEBlbmQKIAogc3RhdGljIHZvaWQgZHVtcFJl bmRlclRyZWUoY29uc3QgY2hhciAqcGF0aE9yVVJMKQo=