48811 2010-11-01 17:47:32 -0700 WebKit2: Repeated cmd-key presses lead to unusable UI state 2010-11-02 10:56:22 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED InRadar P2 Normal --- 1 enrica enrica ap sullivan webkit.review.bot oldest_to_newest 303117 0 enrica 2010-11-01 17:47:32 -0700 To reproduce: 1. Visit a page with an empty form field in it, or use the degenerately-simple one attached to this bug 2. Click in the empty form field 3. Command-X -- beeps, as expected 4. Command-X again -- doesn't beep 5. Command-X once or twice more After a few of these, the insertion point stops blinking, and you can no longer type into the page. If you're on a real page (e.g. google.com), mousing over links no longer shows hover feedback on the page (the cursor changes to a link cursor, but the underlines etc. don't appear), and clicking on links updates the URL field but does not load the clicked page. 303122 1 72610 enrica 2010-11-01 17:58:46 -0700 Created attachment 72610 Patch 303131 2 webkit.review.bot 2010-11-01 18:01:12 -0700 Attachment 72610 did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1 WebKit2/ChangeLog:6: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:7: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:9: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:10: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:11: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:12: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:13: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:14: Line contains tab character. [whitespace/tab] [5] WebKit2/ChangeLog:17: Line contains tab character. [whitespace/tab] [5] Total errors found: 9 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style. 303477 3 enrica 2010-11-02 10:56:22 -0700 Committed revision 71130. 72610 2010-11-01 17:58:46 -0700 2010-11-01 22:14:04 -0700 Patch cmdx-endless-loop.txt text/plain 2864 enrica SW5kZXg6IFdlYktpdDIvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdDIvQ2hhbmdlTG9n CShyZXZpc2lvbiA3MTA4NCkKKysrIFdlYktpdDIvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMjIgQEAKKzIwMTAtMTEtMDEgIEVucmljYSBDYXN1Y2NpICA8ZW5yaWNhQGFwcGxl LmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBX ZWJLaXQyOiBSZXBlYXRlZCBjbWQta2V5IHByZXNzZXMgbGVhZCB0byB1bnVzYWJsZSBVSSBzdGF0 ZQorCQlodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDg4MTEKKwkJPHJk YXI6Ly9wcm9ibGVtLzg2MTE5MjQ+CisKKwkJR2l2ZW4gdGhlIGFzeW5jaHJvbm91cyBuYXR1cmUg b2YgZXZlcnkgaW50ZXJhY3Rpb24gd2l0aCB0aGUgV2ViUHJvY2VzcywKKwkJd2UgY291bGQgcmUt c2VuZCBhIGtleURvd24gZXZlbnQgdG8gQXBwS2l0IHRoYXQgbWFwcyB0byBhbgorCQlhY3Rpb24g dGhhdCBpcyBjdXJyZW50bHkgZGlzYWJsZWQgYW5kIHRoZXJlZm9yZSB3aWxsIG5vdCB0cmlnZ2Vy IGFuIGFjdGlvbgorCQlidXQgYSBjYWxsIHRvIHRoZSBrZXlEb3duIG1ldGhvZCBvZiB0aGUgTlNW aWV3LiBXZSBuZWVkIHRvIGRldGVjdCB0aGlzCisJCXNpdHVhdGlvbiB0byBhdm9pZCBhbiBpbmZp bml0ZSBsb29wLgorCQkKKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL21hYy9XS1ZpZXcubW06Cisg ICAgICAgICgtW1dLVmlldyBwZXJmb3JtS2V5RXF1aXZhbGVudDpdKTogRGVsYXkgdGhlIHJlbGVh c2Ugb2YgdGhlIGxhc3QgcHJvY2Vzc2VkIGV2ZW50CisJCXRvIHRoZSBtb21lbnQgd2UgcmVjZWl2 ZSBhbm90aGVyIGV2ZW50LgorICAgICAgICAoLVtXS1ZpZXcga2V5RG93bjpdKTogQWRkZWQgZWFy bHkgcmV0dXJuIGluIGNhc2UgdGhlIGV2ZW50IGlzIG9uZSB3ZSBoYXZlIHJlLXNlbnQuCisKIDIw MTAtMTEtMDEgIEFuZGVycyBDYXJsc3NvbiAgPGFuZGVyc2NhQGFwcGxlLmNvbT4KIAogICAgICAg ICBSZXZpZXdlZCBieSBPbGl2ZXIgSHVudC4KSW5kZXg6IFdlYktpdDIvVUlQcm9jZXNzL0FQSS9t YWMvV0tWaWV3Lm1tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdDIvVUlQcm9jZXNzL0FQSS9tYWMvV0tW aWV3Lm1tCShyZXZpc2lvbiA3MTAzMSkKKysrIFdlYktpdDIvVUlQcm9jZXNzL0FQSS9tYWMvV0tW aWV3Lm1tCSh3b3JraW5nIGNvcHkpCkBAIC0zNDYsOSArMzQ2LDYgQEAgLSAoQk9PTClwZXJmb3Jt S2V5RXF1aXZhbGVudDooTlNFdmVudCAqKQogICAgIEJPT0wgZXZlbnRXYXNTZW50VG9XZWJDb3Jl ID0gKF9kYXRhLT5fa2V5RG93bkV2ZW50QmVpbmdSZXNlbnQgPT0gZXZlbnQpOwogICAgIEJPT0wg cmV0ID0gTk87CiAgICAgCi0gICAgW19kYXRhLT5fa2V5RG93bkV2ZW50QmVpbmdSZXNlbnQgcmVs ZWFzZV07Ci0gICAgX2RhdGEtPl9rZXlEb3duRXZlbnRCZWluZ1Jlc2VudCA9IG5pbDsKLSAgICAK ICAgICBbc2VsZiByZXRhaW5dOwogICAgIAogICAgIC8vIFBhc3Mga2V5IGNvbWJvcyB0aHJvdWdo IFdlYkNvcmUgaWYgdGhlcmUgaXMgYSBrZXkgYmluZGluZyBhdmFpbGFibGUgZm9yCkBAIC0zNTYs NiArMzUzLDkgQEAgLSAoQk9PTClwZXJmb3JtS2V5RXF1aXZhbGVudDooTlNFdmVudCAqKQogICAg IC8vIEJ1dCBkb24ndCBkbyBpdCBpZiB3ZSBoYXZlIGFscmVhZHkgaGFuZGxlZCB0aGUgZXZlbnQu CiAgICAgLy8gUHJlc3NpbmcgRXNjIHJlc3VsdHMgaW4gYSBmYWtlIGV2ZW50IGJlaW5nIHNlbnQg LSBkb24ndCBwYXNzIGl0IHRvIFdlYkNvcmUuCiAgICAgaWYgKCFldmVudFdhc1NlbnRUb1dlYkNv cmUgJiYgZXZlbnQgPT0gW05TQXBwIGN1cnJlbnRFdmVudF0gJiYgc2VsZiA9PSBbW3NlbGYgd2lu ZG93XSBmaXJzdFJlc3BvbmRlcl0pIHsKKyAgICAgICAgW19kYXRhLT5fa2V5RG93bkV2ZW50QmVp bmdSZXNlbnQgcmVsZWFzZV07CisgICAgICAgIF9kYXRhLT5fa2V5RG93bkV2ZW50QmVpbmdSZXNl bnQgPSBuaWw7CisgICAgICAgIAogICAgICAgICBfZGF0YS0+X3BhZ2UtPmhhbmRsZUtleWJvYXJk RXZlbnQoTmF0aXZlV2ViS2V5Ym9hcmRFdmVudChldmVudCwgc2VsZikpOwogICAgICAgICByZXR1 cm4gWUVTOwogICAgIH0KQEAgLTM4OSw2ICszODksMTUgQEAgLSAodm9pZClrZXlVcDooTlNFdmVu dCAqKXRoZUV2ZW50CiAKIC0gKHZvaWQpa2V5RG93bjooTlNFdmVudCAqKXRoZUV2ZW50CiB7Cisg ICAgLy8gV2UgY291bGQgYmUgcmVjZWl2aW5nIGEga2V5IGRvd24gZnJvbSBBcHBLaXQgaWYgd2Ug aGF2ZSByZS1zZW50IGFuIGV2ZW50CisgICAgLy8gdGhhdCBtYXBzIHRvIGFuIGFjdGlvbiB0aGF0 IGlzIGN1cnJlbnRseSB1bmF2YWlsYWJsZSAoZm9yIGV4YW1wbGUgYSBjb3B5IHdoZW4KKyAgICAv LyB0aGVyZSBpcyBubyByYW5nZSBzZWxlY3Rpb24pLgorICAgIC8vIElmIHRoaXMgaXMgdGhlIGNh c2Ugd2Ugc2hvdWxkIGlnbm9yZSB0aGUga2V5IGRvd24uCisgICAgaWYgKF9kYXRhLT5fa2V5RG93 bkV2ZW50QmVpbmdSZXNlbnQgPT0gdGhlRXZlbnQpIHsKKyAgICAgICAgW19kYXRhLT5fa2V5RG93 bkV2ZW50QmVpbmdSZXNlbnQgcmVsZWFzZV07CisgICAgICAgIF9kYXRhLT5fa2V5RG93bkV2ZW50 QmVpbmdSZXNlbnQgPSBuaWw7CisgICAgICAgIHJldHVybjsKKyAgICB9CiAgICAgX2RhdGEtPl9w YWdlLT5oYW5kbGVLZXlib2FyZEV2ZW50KE5hdGl2ZVdlYktleWJvYXJkRXZlbnQodGhlRXZlbnQs IHNlbGYpKTsKIH0KIAo=