5923 2005-12-03 03:49:55 -0800 REGRESSION: scrolling on Google maps continues after the mouse is released outside the window 2019-02-06 09:03:59 -0800 1 1 1 Unclassified WebKit DOM 420+ Mac OS X 10.4 RESOLVED FIXED http://maps.google.com InRadar, Regression P1 Normal --- 1 ap webkit-unassigned alice.barraclough cdumez darin mitz oldest_to_newest 25429 0 ap 2005-12-03 03:49:55 -0800 Mouse-up events that happen when the mouse is outside a browser window seem to not be delivered. Steps to reproduce: 1. Go to http://maps.google.com 2. Click the mouse anywhere on the mouse, drag until the mouse pointer goes outside the Safari window 3. Release the mouse button 4. Move the mouse pointer back into the window Results: the map scrolls, although the mouse button is not down Expected results (as in Safari 2.0.2): it shouldn't 25430 1 ap 2005-12-03 03:54:09 -0800 Appears related to bug 4822. 28186 2 alice.barraclough 2006-01-10 14:26:00 -0800 <rdar://problem/4404596> 29352 3 joost 2006-01-22 04:36:10 -0800 Adding Regression keyword. 33411 4 darin 2006-02-19 13:05:18 -0800 I'm guessing that either a mouseup or mouseout event is not being delivered that's supposed to be. According to my reading of the DOM specification, the mouseup event goes to whatever element you're over when the mouse goes up. So what if you're not over any element at all (outside the window)? 33443 5 mitz 2006-02-19 22:36:44 -0800 (In reply to comment #4) > I'm guessing that either a mouseup or mouseout event is not being delivered > that's supposed to be. According to my reading of the DOM specification, the > mouseup event goes to whatever element you're over when the mouse goes up. So > what if you're not over any element at all (outside the window)? > The bug could be on a lower level, if it is related to another thing I observed when leaving the window during a drag operation: if you start dragging the separator between two frames and leave the window, the separator stops following the mouse, and remains active even if you release the mouse button. If you return to the window, it starts tracking the mouse again, even if the button is up. 33499 6 mitz 2006-02-20 09:23:06 -0800 Yes, here, too, the problem show already when the mouse leaves the window and the map stops following it. In shipping Safari, the map keeps moving even when the mouse is outside the window for as long as the button is pressed. 34978 7 mitz 2006-03-03 13:26:02 -0800 I tracked this bug down to this change, from r11105 (Fix hit testing of relative positioned inline flows inside clipped overflow blocks so that they aren't mistakenly included in hit testing): bool RenderLayer::containsPoint(int x, int y, const QRect& damageRect) const { - return mustExamineRenderer(renderer()) || damageRect.contains(x, y); + return damageRect.contains(x, y); } Undoing this change alone fixes the regression (and the frame-resize separator issue too). 35742 8 6987 mitz 2006-03-10 10:33:51 -0800 Created attachment 6987 Patch 36030 9 6987 mjs 2006-03-12 12:30:12 -0800 Comment on attachment 6987 Patch r=me 1503079 10 lforschler 2019-02-06 09:03:59 -0800 Mass moving XML DOM bugs to the "DOM" Component. 6987 2006-03-10 10:33:51 -0800 2006-03-12 12:30:12 -0800 Patch 5923_r1.patch text/plain 4778 mitz SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDEzMjQ1KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMDYtMDMtIyMgIE1pdHogUGV0dGVsICA8b3Bl bmRhcndpbi5vcmdAbWl0enBldHRlbC5jb20+CisKKyAgICAgICAgLSB0ZXN0IGZvciBodHRwOi8v YnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lkPTU5MjMKKyAgICAgICAgICBS RUdSRVNTSU9OOiBzY3JvbGxpbmcgb24gR29vZ2xlIG1hcHMgY29udGludWVzIGFmdGVyIHRoZSBt b3VzZSBpcyByZWxlYXNlZCBvdXRzaWRlIHRoZSB3aW5kb3cKKworICAgICAgICBSZXZpZXdlZCBi eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGZhc3QvZXZlbnRzL2RyYWctb3V0c2lkZS13 aW5kb3ctZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2V2ZW50cy9kcmFnLW91 dHNpZGUtd2luZG93Lmh0bWw6IEFkZGVkLgorICAgICAgICAqIGZhc3QvZXZlbnRzL3Jlc291cmNl czogQWRkZWQuCisgICAgICAgICogZmFzdC9ldmVudHMvcmVzb3VyY2VzL2RyYWctb3V0c2lkZS13 aW5kb3ctZnJhbWUuaHRtbDogQWRkZWQuCisKIDIwMDYtMDMtMDkgIEFkZWxlIFBldGVyc29uICA8 YWRlbGVAYXBwbGUuY29tPgogCiAgICAgICAgIC0gVGVzdCBmb3IgaHR0cDovL2J1Z3ppbGxhLm9w ZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD03Njc1CkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0 L2V2ZW50cy9kcmFnLW91dHNpZGUtd2luZG93LWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBM YXlvdXRUZXN0cy9mYXN0L2V2ZW50cy9kcmFnLW91dHNpZGUtd2luZG93LWV4cGVjdGVkLnR4dAko cmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvZXZlbnRzL2RyYWctb3V0c2lkZS13aW5k b3ctZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDUgQEAKK0VESVRJTkcgREVM RUdBVEU6IHNob3VsZENoYW5nZVNlbGVjdGVkRE9NUmFuZ2U6KG51bGwpIHRvRE9NUmFuZ2U6cmFu Z2UgZnJvbSAzMSBvZiAjdGV4dCA+IFAgPiBCT0RZID4gSFRNTCA+ICNkb2N1bWVudCB0byAzMSBv ZiAjdGV4dCA+IFAgPiBCT0RZID4gSFRNTCA+ICNkb2N1bWVudCBhZmZpbml0eTpOU1NlbGVjdGlv bkFmZmluaXR5RG93bnN0cmVhbSBzdGlsbFNlbGVjdGluZzpGQUxTRQorRURJVElORyBERUxFR0FU RTogd2ViVmlld0RpZENoYW5nZVNlbGVjdGlvbjpXZWJWaWV3RGlkQ2hhbmdlU2VsZWN0aW9uTm90 aWZpY2F0aW9uCitFRElUSU5HIERFTEVHQVRFOiBzaG91bGRDaGFuZ2VTZWxlY3RlZERPTVJhbmdl OnJhbmdlIGZyb20gMzEgb2YgI3RleHQgPiBQID4gQk9EWSA+IEhUTUwgPiAjZG9jdW1lbnQgdG8g MzEgb2YgI3RleHQgPiBQID4gQk9EWSA+IEhUTUwgPiAjZG9jdW1lbnQgdG9ET01SYW5nZTpyYW5n ZSBmcm9tIDMxIG9mICN0ZXh0ID4gUCA+IEJPRFkgPiBIVE1MID4gI2RvY3VtZW50IHRvIDMxIG9m ICN0ZXh0ID4gUCA+IEJPRFkgPiBIVE1MID4gI2RvY3VtZW50IGFmZmluaXR5Ok5TU2VsZWN0aW9u QWZmaW5pdHlEb3duc3RyZWFtIHN0aWxsU2VsZWN0aW5nOkZBTFNFCitBTEVSVDogVGVzdCByZXN1 bHQ6IFBBU1MKKwpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9ldmVudHMvZHJhZy1vdXRzaWRlLXdp bmRvdy5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvZXZlbnRzL2RyYWctb3V0 c2lkZS13aW5kb3cuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvZXZlbnRz L2RyYWctb3V0c2lkZS13aW5kb3cuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSw0IEBACis8 ZnJhbWVzZXQgY29scz0iMTAwLCAqIj4KKzxmcmFtZT4KKzxmcmFtZSBzcmM9InJlc291cmNlcy9k cmFnLW91dHNpZGUtd2luZG93LWZyYW1lLmh0bWwiPgorPC9mcmFtZXNldD4KSW5kZXg6IExheW91 dFRlc3RzL2Zhc3QvZXZlbnRzL3Jlc291cmNlcy9kcmFnLW91dHNpZGUtd2luZG93LWZyYW1lLmh0 bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9ldmVudHMvcmVzb3VyY2VzL2RyYWct b3V0c2lkZS13aW5kb3ctZnJhbWUuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zh c3QvZXZlbnRzL3Jlc291cmNlcy9kcmFnLW91dHNpZGUtd2luZG93LWZyYW1lLmh0bWwJKHJldmlz aW9uIDApCkBAIC0wLDAgKzEsMzEgQEAKKzxib2R5IG9ubG9hZD0idGVzdCgpIj4KKzxzY3JpcHQg dHlwZT0idGV4dC9qYXZhc2NyaXB0Ij4KK2RvY3VtZW50LmFkZEV2ZW50TGlzdGVuZXIoIm1vdXNl bW92ZSIsIG1vdXNlbW92ZWQpOworCit2YXIgcmVzdWx0ID0gIkZBSUwiOworCitmdW5jdGlvbiBt b3VzZW1vdmVkKGV2ZW50KQoreworICAgIGlmIChldmVudC5jbGllbnRYIDwgLTEpCisgICAgICAg IHJlc3VsdCA9ICJQQVNTIjsKK30KKworZnVuY3Rpb24gdGVzdCgpCit7CisgICAgaWYgKCF3aW5k b3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisgICAgICAgIHJldHVybjsKKyAgICBsYXlvdXRUZXN0 Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7CisgICAgZXZlbnRTZW5kZXIubW91c2VNb3ZlVG8oMTUw LCAxMDApOworICAgIGV2ZW50U2VuZGVyLm1vdXNlRG93bigpOworICAgIGV2ZW50U2VuZGVyLm1v dXNlTW92ZVRvKDUwLCAxMDApOworICAgIGFsZXJ0KCJUZXN0IHJlc3VsdDogIiArIHJlc3VsdCk7 Cit9Cis8L3NjcmlwdD4KKzxwPgorICAgIFRoaXMgaXMgYSB0ZXN0IGZvciA8aT48YSBocmVmPSJo dHRwOi8vYnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lkPTU5MjMiPmh0dHA6 Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTkyMzwvYT4KKyAgICBS RUdSRVNTSU9OOiBzY3JvbGxpbmcgb24gR29vZ2xlIG1hcHMgY29udGludWVzIGFmdGVyIHRoZSBt b3VzZSBpcyByZWxlYXNlZCBvdXRzaWRlIHRoZSB3aW5kb3c8L2k+LgorPC9wPgorPHA+CisgICAg KENhbm5vdCBydW4gaW50ZXJhY3RpdmVseSkKKzwvcD4KKzwvYm9keT4KSW5kZXg6IFdlYkNvcmUv Q2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9nCShyZXZpc2lvbiAxMzI0 NSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAK KzIwMDYtMDMtIyMgIE1pdHogUGV0dGVsICA8b3BlbmRhcndpbi5vcmdAbWl0enBldHRlbC5jb20+ CisKKyAgICAgICAgVGVzdDogZmFzdC9ldmVudHMvZHJhZy1vdXRzaWRlLXdpbmRvdy5odG1sCisK KyAgICAgICAgLSBmaXggaHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNn aT9pZD01OTIzCisgICAgICAgICAgUkVHUkVTU0lPTjogc2Nyb2xsaW5nIG9uIEdvb2dsZSBtYXBz IGNvbnRpbnVlcyBhZnRlciB0aGUgbW91c2UgaXMgcmVsZWFzZWQgb3V0c2lkZSB0aGUgd2luZG93 CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiByZW5k ZXJpbmcvcmVuZGVyX2xheWVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlJlbmRlckxheWVyOjpj b250YWluc1BvaW50KTogQWx3YXlzIHJldHVybiB0cnVlIGZvciB0aGUgcm9vdAorICAgICAgICBv YmplY3QgdG8gZW5zdXJlIHRoYXQgbW91c2UgZXZlbnRzIG9jY3VycmluZyBvdXRzaWRlIHRoZSB3 aW5kb3cgdGFyZ2V0CisgICAgICAgIHRoZSBkb2N1bWVudC4KKwogMjAwNi0wMy0wOSAgSnVzdGlu IEdhcmNpYSAgPGp1c3Rpbi5nYXJjaWFAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5 IGFkZWxlCkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9yZW5kZXJfbGF5ZXIuY3BwCj09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT0KLS0tIFdlYkNvcmUvcmVuZGVyaW5nL3JlbmRlcl9sYXllci5jcHAJKHJldmlzaW9uIDEzMjQ0 KQorKysgV2ViQ29yZS9yZW5kZXJpbmcvcmVuZGVyX2xheWVyLmNwcAkod29ya2luZyBjb3B5KQpA QCAtMTMxMyw3ICsxMzEzLDkgQEAgYm9vbCBSZW5kZXJMYXllcjo6aW50ZXJzZWN0c0RhbWFnZVJl Y3QoYwogCiBib29sIFJlbmRlckxheWVyOjpjb250YWluc1BvaW50KGludCB4LCBpbnQgeSwgY29u c3QgSW50UmVjdCYgZGFtYWdlUmVjdCkgY29uc3QKIHsKLSAgICByZXR1cm4gZGFtYWdlUmVjdC5j b250YWlucyh4LCB5KTsKKyAgICAvLyBBbHdheXMgcmV0dXJuaW5nIHRydWUgZm9yIHRoZSByb290 IG9iamVjdCB0byBlbnN1cmUgdGhhdCBtb3VzZSBldmVudHMgb2NjdXJyaW5nCisgICAgLy8gb3V0 c2lkZSB0aGUgd2luZG93ICh3aGVuIGRyYWdnaW5nKSBhbHdheXMgdGFyZ2V0IHNvbWUgbm9kZS4K KyAgICByZXR1cm4gcmVuZGVyZXIoKS0+aXNSb290KCkgfHwgZGFtYWdlUmVjdC5jb250YWlucyh4 LCB5KTsKIH0KIAogdm9pZCBSZW5kZXJMYXllcjo6Y2xlYXJDbGlwUmVjdHMoKQo=