14495 2007-07-02 03:37:01 -0700 REGRESSION: Wrong cursor is displayed during drag and drop when text is selected 2007-07-04 03:57:30 -0700 1 1 1 Unclassified WebKit CSS 523.x (Safari 3) All All RESOLVED FIXED InRadar, Regression P1 Normal --- 1 mjuhos webkit-unassigned mitz oldest_to_newest 5747 0 mjuhos 2007-07-02 03:37:01 -0700 Overview Description: Wrong cursor is displayed during drag and drop when text is selected Steps to Reproduce: 1) Open test case 2) Select text in yellow box 3) Move using drag'n'drop blue box Actual Results: Text cursor is displayed as mouse cursor. Expected Results: Mouse cursor will be "default" thus arrow Build Date & Platform: WebKit r23922 on WinXP Safari 3.0 (5222.8.3) on Mac OS 10.5. (9A466) Additional Information: On Safari 2.0.4 (419.3) test case works correctly. Test case: <html> <style> * { cursor:default; } div { position:absolute; width:100px; height:100px; } </style> <script> var allowMove = false; var movable; onload = function() { movable = document.getElementById('movable'); movable.onmousedown = function() { allowMove = true; return false; } } onmouseup = function(event) {allowMove = false;} onmousemove = function(event) { if (allowMove) { movable.style.top = event.clientY + 10 + 'px'; movable.style.left = event.clientX + 10 + 'px'; } } </script> <body> <div style="background-color:blue;left:100px;position:absolute" id="movable"></div> <div style="background-color:yellow;left:300px;">Select me</div> </body> </html> 5748 1 15347 mjuhos 2007-07-02 03:37:51 -0700 Created attachment 15347 Test case 5750 2 ap 2007-07-02 03:56:42 -0700 Confirmed as a regression with r23841. 5645 3 15372 mitz 2007-07-03 12:04:37 -0700 Created attachment 15372 Don't change the cursor to an I-beam unconditionally if selection is impossible 5608 4 15372 mjs 2007-07-04 03:30:39 -0700 Comment on attachment 15372 Don't change the cursor to an I-beam unconditionally if selection is impossible r=me 5613 5 mrowe 2007-07-04 03:55:50 -0700 <rdar://problem/5312384> 5614 6 mrowe 2007-07-04 03:57:30 -0700 Landed in r23983. 15347 2007-07-02 03:37:51 -0700 2007-07-02 03:37:51 -0700 Test case cursor.html text/html 773 mjuhos PGh0bWw+DQoJPHN0eWxlPg0KCQkqIHsNCgkJCWN1cnNvcjpkZWZhdWx0Ow0KCQl9DQoJCWRpdiB7 DQoJCQlwb3NpdGlvbjphYnNvbHV0ZTsNCgkJCXdpZHRoOjEwMHB4Ow0KCQkJaGVpZ2h0OjEwMHB4 Ow0KCQl9DQoJPC9zdHlsZT4NCgk8c2NyaXB0Pg0KCQl2YXIgYWxsb3dNb3ZlID0gZmFsc2U7DQoJ CXZhciBtb3ZhYmxlOw0KCQlvbmxvYWQgPSBmdW5jdGlvbigpIHsNCgkJCW1vdmFibGUgPSBkb2N1 bWVudC5nZXRFbGVtZW50QnlJZCgnbW92YWJsZScpOw0KCQkJbW92YWJsZS5vbm1vdXNlZG93biA9 IGZ1bmN0aW9uKCkgew0KCQkJCWFsbG93TW92ZSA9IHRydWU7DQoJCQkJcmV0dXJuIGZhbHNlOw0K CQkJfQ0KCQl9DQoJCW9ubW91c2V1cCA9IGZ1bmN0aW9uKGV2ZW50KSB7YWxsb3dNb3ZlID0gZmFs c2U7fQ0KCQlvbm1vdXNlbW92ZSA9IGZ1bmN0aW9uKGV2ZW50KSB7DQoJCQlpZiAoYWxsb3dNb3Zl KSB7DQoJCQkJbW92YWJsZS5zdHlsZS50b3AgPSBldmVudC5jbGllbnRZICArIDEwICsgJ3B4JzsN CgkJCQltb3ZhYmxlLnN0eWxlLmxlZnQgPSBldmVudC5jbGllbnRYICsgMTAgKyAncHgnOw0KCQkJ fQ0KCQl9DQoJPC9zY3JpcHQ+DQoJPGJvZHk+DQoJCTxkaXYgc3R5bGU9ImJhY2tncm91bmQtY29s b3I6Ymx1ZTtsZWZ0OjEwMHB4O3Bvc2l0aW9uOmFic29sdXRlIiBpZD0ibW92YWJsZSI+PC9kaXY+ DQoJCTxkaXYgc3R5bGU9ImJhY2tncm91bmQtY29sb3I6eWVsbG93O2xlZnQ6MzAwcHg7Ij5TZWxl Y3QgbWU8L2Rpdj4NCgk8L2JvZHk+DQo8L2h0bWw+DQo= 15372 2007-07-03 12:04:37 -0700 2007-07-04 03:30:39 -0700 Don't change the cursor to an I-beam unconditionally if selection is impossible 11495_r1.patch text/plain 5218 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAyMzk0NykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTkgQEAKKzIwMDctMDctMDMgIE1pdHogUGV0dGVsICA8bWl0ekB3ZWJraXQub3Jn PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIC0gZml4 IGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0NDk1CisgICAgICAgICAg UkVHUkVTU0lPTjogV3JvbmcgY3Vyc29yIGlzIGRpc3BsYXllZCBkdXJpbmcgZHJhZyBhbmQgZHJv cCB3aGVuIHRleHQgaXMgc2VsZWN0ZWQKKworICAgICAgICBObyB0ZXN0IHBvc3NpYmxlIGJlY2F1 c2UgRHVtcFJlbmRlclRyZWUgY2Fubm90IHF1ZXJ5IHRoZSBjdXJyZW50IGN1cnNvci4KKworICAg ICAgICAqIHBhZ2UvRXZlbnRIYW5kbGVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkV2ZW50SGFu ZGxlcjo6c2VsZWN0Q3Vyc29yKTogQ2hhbmdlZCBpbnRvIGEgbWVtYmVyIGZ1bmN0aW9uIGFuZAor ICAgICAgICBhZGRlZCB0aGF0IGlmIHRoZSBsYXN0IG1vdXNlIGRvd24gZXZlbnQgY291bGQgbm90 IGhhdmUgc3RhcnRlZCBhCisgICAgICAgIHNlbGVjdGlvbiwgdGhlbiB0aGUgY3Vyc29yIHNob3Vs ZCBub3QgY2hhbmdlIHRvIGFuIEktYmVhbSB1bmNvbmRpdGlvbmFsbHkuCisgICAgICAgIChXZWJD b3JlOjpFdmVudEhhbmRsZXI6OmhhbmRsZU1vdXNlTW92ZUV2ZW50KToKKyAgICAgICAgKiBwYWdl L0V2ZW50SGFuZGxlci5oOgorCiAyMDA3LTA3LTAzICBBbmRlcnMgQ2FybHNzb24gIDxhbmRlcnNj YUBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQWRhbS4KSW5kZXg6IFdlYkNvcmUv cGFnZS9FdmVudEhhbmRsZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvcGFnZS9FdmVudEhh bmRsZXIuY3BwCShyZXZpc2lvbiAyMzk0MSkKKysrIFdlYkNvcmUvcGFnZS9FdmVudEhhbmRsZXIu Y3BwCSh3b3JraW5nIGNvcHkpCkBAIC02MzYsMTIgKzYzNiwxMSBAQCBzdGF0aWMgYm9vbCBub2Rl SXNOb3RCZWluZ0VkaXRlZChOb2RlKiBuCiAgICAgcmV0dXJuIGZyYW1lLT5zZWxlY3Rpb25Db250 cm9sbGVyKCktPnJvb3RFZGl0YWJsZUVsZW1lbnQoKSAhPSBub2RlLT5yb290RWRpdGFibGVFbGVt ZW50KCk7CiB9CiAKLS8vIEZJWE1FOiBXZSBzaG91bGQgY29uc2lkZXIgbWFraW5nIHRoaXMgYSBt ZW1iZXIgZnVuY3Rpb24gbm93IHRoYXQgd2UncmUgcGFzc2luZyBpbiBtb3JlIGluZm8gYWJvdXQg dGhlIG9iamVjdC4KLXN0YXRpYyBDdXJzb3Igc2VsZWN0Q3Vyc29yKGNvbnN0IE1vdXNlRXZlbnRX aXRoSGl0VGVzdFJlc3VsdHMmIGV2ZW50LCBGcmFtZSogZnJhbWUsIGJvb2wgbW91c2VQcmVzc2Vk LCBQbGF0Zm9ybVNjcm9sbGJhciogc2Nyb2xsYmFyLCBib29sIGNhcHR1cmluZ01vdXNlRXZlbnRz KQorQ3Vyc29yIEV2ZW50SGFuZGxlcjo6c2VsZWN0Q3Vyc29yKGNvbnN0IE1vdXNlRXZlbnRXaXRo SGl0VGVzdFJlc3VsdHMmIGV2ZW50LCBQbGF0Zm9ybVNjcm9sbGJhciogc2Nyb2xsYmFyKQogewog ICAgIC8vIER1cmluZyBzZWxlY3Rpb24sIHVzZSBhbiBJLWJlYW0gbm8gbWF0dGVyIHdoYXQgd2Un cmUgb3Zlci4KICAgICAvLyBJZiB5b3UncmUgY2FwdHVyaW5nIG1vdXNlIGV2ZW50cyBmb3IgYSBw YXJ0aWN1bGFyIG5vZGUsIGRvbid0IHRyZWF0IHRoaXMgYXMgYSBzZWxlY3Rpb24uCi0gICAgaWYg KG1vdXNlUHJlc3NlZCAmJiBmcmFtZS0+c2VsZWN0aW9uQ29udHJvbGxlcigpLT5pc0NhcmV0T3JS YW5nZSgpICYmICFjYXB0dXJpbmdNb3VzZUV2ZW50cykKKyAgICBpZiAobV9tb3VzZVByZXNzZWQg JiYgbV9tb3VzZURvd25NYXlTdGFydFNlbGVjdCAmJiBtX2ZyYW1lLT5zZWxlY3Rpb25Db250cm9s bGVyKCktPmlzQ2FyZXRPclJhbmdlKCkgJiYgIW1fY2FwdHVyaW5nTW91c2VFdmVudHNOb2RlKQog ICAgICAgICByZXR1cm4gaUJlYW1DdXJzb3IoKTsKIAogICAgIE5vZGUqIG5vZGUgPSBldmVudC50 YXJnZXROb2RlKCk7CkBAIC02NzksOCArNjc4LDggQEAgc3RhdGljIEN1cnNvciBzZWxlY3RDdXJz b3IoY29uc3QgTW91c2VFdgogCiAgICAgICAgICAgICAvLyBJZiB0aGUgbGluayBpcyBlZGl0YWJs ZSwgdGhlbiB3ZSBuZWVkIHRvIGNoZWNrIHRoZSBzZXR0aW5ncyB0byBzZWUgd2hldGhlciBvciBu b3QgdGhlIGxpbmsgc2hvdWxkIGJlIGZvbGxvd2VkCiAgICAgICAgICAgICBpZiAoZWRpdGFibGUp IHsKLSAgICAgICAgICAgICAgICBBU1NFUlQoZnJhbWUtPnNldHRpbmdzKCkpOwotICAgICAgICAg ICAgICAgIHN3aXRjaChmcmFtZS0+c2V0dGluZ3MoKS0+ZWRpdGFibGVMaW5rQmVoYXZpb3IoKSkg eworICAgICAgICAgICAgICAgIEFTU0VSVChtX2ZyYW1lLT5zZXR0aW5ncygpKTsKKyAgICAgICAg ICAgICAgICBzd2l0Y2gobV9mcmFtZS0+c2V0dGluZ3MoKS0+ZWRpdGFibGVMaW5rQmVoYXZpb3Io KSkgewogICAgICAgICAgICAgICAgICAgICBkZWZhdWx0OgogICAgICAgICAgICAgICAgICAgICBj YXNlIEVkaXRhYmxlTGlua0RlZmF1bHRCZWhhdmlvcjoKICAgICAgICAgICAgICAgICAgICAgY2Fz ZSBFZGl0YWJsZUxpbmtBbHdheXNMaXZlOgpAQCAtNjkyLDcgKzY5MSw3IEBAIHN0YXRpYyBDdXJz b3Igc2VsZWN0Q3Vyc29yKGNvbnN0IE1vdXNlRXYKICAgICAgICAgICAgICAgICAgICAgICAgIGJy ZWFrOwogCiAgICAgICAgICAgICAgICAgICAgIGNhc2UgRWRpdGFibGVMaW5rTGl2ZVdoZW5Ob3RG b2N1c2VkOgotICAgICAgICAgICAgICAgICAgICAgICAgZWRpdGFibGVMaW5rRW5hYmxlZCA9IG5v ZGVJc05vdEJlaW5nRWRpdGVkKG5vZGUsIGZyYW1lKSB8fCBldmVudC5ldmVudCgpLnNoaWZ0S2V5 KCk7CisgICAgICAgICAgICAgICAgICAgICAgICBlZGl0YWJsZUxpbmtFbmFibGVkID0gbm9kZUlz Tm90QmVpbmdFZGl0ZWQobm9kZSwgbV9mcmFtZSkgfHwgZXZlbnQuZXZlbnQoKS5zaGlmdEtleSgp OwogICAgICAgICAgICAgICAgICAgICAgICAgYnJlYWs7CiAgICAgICAgICAgICAgICAgICAgIAog ICAgICAgICAgICAgICAgICAgICBjYXNlIEVkaXRhYmxlTGlua09ubHlMaXZlV2l0aFNoaWZ0S2V5 OgpAQCAtNzA1LDcgKzcwNCw3IEBAIHN0YXRpYyBDdXJzb3Igc2VsZWN0Q3Vyc29yKGNvbnN0IE1v dXNlRXYKICAgICAgICAgICAgICAgICByZXR1cm4gaGFuZEN1cnNvcigpOwogICAgICAgICAgICAg UmVuZGVyTGF5ZXIqIGxheWVyID0gcmVuZGVyZXIgPyByZW5kZXJlci0+ZW5jbG9zaW5nTGF5ZXIo KSA6IDA7CiAgICAgICAgICAgICBib29sIGluUmVzaXplciA9IGZhbHNlOwotICAgICAgICAgICAg aWYgKGZyYW1lLT52aWV3KCkgJiYgbGF5ZXIgJiYgbGF5ZXItPmlzUG9pbnRJblJlc2l6ZUNvbnRy b2woZnJhbWUtPnZpZXcoKS0+d2luZG93VG9Db250ZW50cyhldmVudC5ldmVudCgpLnBvcygpKSkp CisgICAgICAgICAgICBpZiAobV9mcmFtZS0+dmlldygpICYmIGxheWVyICYmIGxheWVyLT5pc1Bv aW50SW5SZXNpemVDb250cm9sKG1fZnJhbWUtPnZpZXcoKS0+d2luZG93VG9Db250ZW50cyhldmVu dC5ldmVudCgpLnBvcygpKSkpCiAgICAgICAgICAgICAgICAgaW5SZXNpemVyID0gdHJ1ZTsKICAg ICAgICAgICAgIGlmICgoZWRpdGFibGUgfHwgKHJlbmRlcmVyICYmIHJlbmRlcmVyLT5pc1RleHQo KSAmJiByZW5kZXJlci0+Y2FuU2VsZWN0KCkpKSAmJiAhaW5SZXNpemVyICYmICFzY3JvbGxiYXIp CiAgICAgICAgICAgICAgICAgcmV0dXJuIGlCZWFtQ3Vyc29yKCk7CkBAIC05NjIsNyArOTYxLDcg QEAgYm9vbCBFdmVudEhhbmRsZXI6OmhhbmRsZU1vdXNlTW92ZUV2ZW50KAogICAgICAgICBpZiAo c2Nyb2xsYmFyICYmICFtX21vdXNlUHJlc3NlZCkKICAgICAgICAgICAgIHNjcm9sbGJhci0+aGFu ZGxlTW91c2VNb3ZlRXZlbnQobW91c2VFdmVudCk7IC8vIEhhbmRsZSBob3ZlciBlZmZlY3RzIG9u IHBsYXRmb3JtcyB0aGF0IHN1cHBvcnQgdmlzdWFsIGZlZWRiYWNrIG9uIHNjcm9sbGJhciBob3Zl cmluZy4KICAgICAgICAgaWYgKCghbV9yZXNpemVMYXllciB8fCAhbV9yZXNpemVMYXllci0+aW5S ZXNpemVNb2RlKCkpICYmIG1fZnJhbWUtPnZpZXcoKSkKLSAgICAgICAgICAgIG1fZnJhbWUtPnZp ZXcoKS0+c2V0Q3Vyc29yKHNlbGVjdEN1cnNvcihtZXYsIG1fZnJhbWUsIG1fbW91c2VQcmVzc2Vk LCBzY3JvbGxiYXIsIG1fY2FwdHVyaW5nTW91c2VFdmVudHNOb2RlKSk7CisgICAgICAgICAgICBt X2ZyYW1lLT52aWV3KCktPnNldEN1cnNvcihzZWxlY3RDdXJzb3IobWV2LCBzY3JvbGxiYXIpKTsK ICAgICB9CiAgICAgCiAgICAgbV9sYXN0TW91c2VNb3ZlRXZlbnRTdWJmcmFtZSA9IG5ld1N1YmZy YW1lOwpJbmRleDogV2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxlci5oCj09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdl YkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuaAkocmV2aXNpb24gMjM5NDEpCisrKyBXZWJDb3JlL3Bh Z2UvRXZlbnRIYW5kbGVyLmgJKHdvcmtpbmcgY29weSkKQEAgLTQ4LDYgKzQ4LDcgQEAgbmFtZXNw YWNlIFdlYkNvcmUgewogCiBjbGFzcyBBdG9taWNTdHJpbmc7CiBjbGFzcyBDbGlwYm9hcmQ7Citj bGFzcyBDdXJzb3I7CiBjbGFzcyBFdmVudFRhcmdldE5vZGU7CiBjbGFzcyBFdmVudDsKIGNsYXNz IEZsb2F0UG9pbnQ7CkBAIC0xODcsNiArMTg4LDggQEAgcHJpdmF0ZToKICAgICBib29sIGhhbmRs ZU1vdXNlRHJhZ2dlZEV2ZW50KGNvbnN0IE1vdXNlRXZlbnRXaXRoSGl0VGVzdFJlc3VsdHMmKTsK ICAgICBib29sIGhhbmRsZU1vdXNlUmVsZWFzZUV2ZW50KGNvbnN0IE1vdXNlRXZlbnRXaXRoSGl0 VGVzdFJlc3VsdHMmKTsKIAorICAgIEN1cnNvciBzZWxlY3RDdXJzb3IoY29uc3QgTW91c2VFdmVu dFdpdGhIaXRUZXN0UmVzdWx0cyYsIFBsYXRmb3JtU2Nyb2xsYmFyKik7CisKICAgICB2b2lkIGhv dmVyVGltZXJGaXJlZChUaW1lcjxFdmVudEhhbmRsZXI+Kik7CiAKICAgICBzdGF0aWMgYm9vbCBj YW5Nb3VzZURvd25TdGFydFNlbGVjdChOb2RlKik7Cg==