76408 2012-01-16 16:29:06 -0800 REGRESSION: r102553 Autocorrection bubble doesn't show up 2012-01-16 17:13:26 -0800 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) Mac OS X 10.7 RESOLVED FIXED InRadar P2 Normal --- 1 enrica enrica jiapu.mail oldest_to_newest 536799 0 enrica 2012-01-16 16:29:06 -0800 Typing a correctable word and pausing before hitting space seems to bring up red underline dots rather than showing a bubble. Hitting space and back space doesn't bring up the correction bubble either. The regression has been introduced with http://trac.webkit.org/changeset/102553. 536800 1 enrica 2012-01-16 16:29:34 -0800 <rdar://problem/10679706> 536802 2 122692 enrica 2012-01-16 16:44:50 -0800 Created attachment 122692 Patch 536807 3 122692 darin 2012-01-16 16:55:37 -0800 Comment on attachment 122692 Patch Is there a way we can add a regression test for this? 536812 4 enrica 2012-01-16 17:03:57 -0800 (In reply to comment #3) > (From update of attachment 122692 [details]) > Is there a way we can add a regression test for this? With what we have today the answer is no. We need support from AppKit to know if the bubble is visible or not, and possibly its position. 536818 5 enrica 2012-01-16 17:13:26 -0800 Committed revision 105111. 122692 2012-01-16 16:44:50 -0800 2012-01-16 16:55:37 -0800 Patch bubblenoshow.txt text/plain 1660 enrica SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEwNTEwNSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE5IEBACisyMDEyLTAxLTE2ICBFbnJpY2Eg Q2FzdWNjaSAgPGVucmljYUBhcHBsZS5jb20+CisKKyAgICAgICAgUkVHUkVTU0lPTjogcjEwMjU1 MyBBdXRvY29ycmVjdGlvbiBidWJibGUgZG9lc24ndCBzaG93IHVwCisgICAgICAgIGh0dHBzOi8v YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD03NjQwOAorICAgICAgICA8cmRhcjovL3By b2JsZW0vMTA2NDQ3NDY+CisKKyAgICAgICAgUHJpb3IgdG8gcjEwMjU1MyBFZGl0b3I6Om1hcmtB bGxNaXNzcGVsbGluZ3NBbmRCYWRHcmFtbWFySW5SYW5nZXMoKSB1c2VkIHRvIHBhc3MgaXRzCisg ICAgICAgIHRleHRDaGVja2luZ09wdGlvbnMgZGlyZWN0bHkgdG8gbWFya0FuZFJlcGxhY2VGb3Io KS4gTm93IGEgcmVxdWVzdCBvYmplY3QgaXMgdXNlZCwKKyAgICAgICAgYnV0LCB3aGVuIHRoZSBv YmplY3QgaXMgY3JlYXRlZCwgdGhlIFRleHRDaGVja2luZ1R5cGVTaG93Q29ycmVjdGlvblBhbmVs IGZsYWcgaXMgZHJvcHBlZC4KKyAgICAgICAgCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo T09QUyEpLgorCisgICAgICAgICogZWRpdGluZy9FZGl0b3IuY3BwOgorICAgICAgICAoV2ViQ29y ZTo6RWRpdG9yOjpyZXNvbHZlVGV4dENoZWNraW5nVHlwZU1hc2spOiBBZGQgVGV4dENoZWNraW5n VHlwZVNob3dDb3JyZWN0aW9uUGFuZWwgdG8KKyAgICAgICAgdGhlIHJldHVybmVkIGNoZWNraW5n VHlwZXMgd2hlbiBhcHByb3ByaWF0ZS4KKwogMjAxMi0wMS0xNiAgSm9uIExlZSAgPGpvbmxlZUBh cHBsZS5jb20+CiAKICAgICAgICAgQnVpbGQgZml4IGZvciByMTA1MDg2LgpJbmRleDogU291cmNl L1dlYkNvcmUvZWRpdGluZy9FZGl0b3IuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3Jl L2VkaXRpbmcvRWRpdG9yLmNwcAkocmV2aXNpb24gMTA1MDcxKQorKysgU291cmNlL1dlYkNvcmUv ZWRpdGluZy9FZGl0b3IuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zMDc1LDYgKzMwNzUsOCBAQCBU ZXh0Q2hlY2tpbmdUeXBlTWFzayBFZGl0b3I6OnJlc29sdmVUZXh0CiAgICAgICAgIGNoZWNraW5n VHlwZXMgfD0gVGV4dENoZWNraW5nVHlwZUdyYW1tYXI7CiAgICAgaWYgKHNob3VsZENoZWNrRm9y Q29ycmVjdGlvbikKICAgICAgICAgY2hlY2tpbmdUeXBlcyB8PSBUZXh0Q2hlY2tpbmdUeXBlQ29y cmVjdGlvbjsKKyAgICBpZiAoc2hvdWxkU2hvd0NvcnJlY3Rpb25QYW5lbCkKKyAgICAgICAgY2hl Y2tpbmdUeXBlcyB8PSBUZXh0Q2hlY2tpbmdUeXBlU2hvd0NvcnJlY3Rpb25QYW5lbDsKIAogI2lm IFVTRShBVVRPTUFUSUNfVEVYVF9SRVBMQUNFTUVOVCkKICAgICBib29sIHNob3VsZFBlcmZvcm1S ZXBsYWNlbWVudCA9IHRleHRDaGVja2luZ09wdGlvbnMgJiBUZXh0Q2hlY2tpbmdUeXBlUmVwbGFj ZW1lbnQ7Cg==