82079 2012-03-23 12:35:16 -0700 REGRESSION: editing/inserting/insert-bg-font.html fails on Lion 2012-04-30 19:08:32 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P1 Normal --- 1 rniwa webkit-unassigned darin enrica jberlin mitz webkit.review.bot oldest_to_newest 586614 0 rniwa 2012-03-23 12:35:16 -0700 It seems like editing/inserting/insert-bg-font.html has been failing on Lion bots. http://build.webkit.org/results/Lion%20Intel%20Release%20(Tests)/r111885%20(6782)/results.html --- /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/editing/inserting/insert-bg-font-expected.txt +++ /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/editing/inserting/insert-bg-font-actual.txt @@ -1,5 +1,5 @@ - <div id="dv" contenteditable=""><span style="background-color: lightgrey">lightgrey background<font color="blue">blue font color over lightgrey background</font></span> + <div id="dv" contenteditable=""><span style="background-color: lightgrey">lightgrey&nbsp;background blue<font color="#0000ff">&nbsp;font color over lightgrey background</font></span> <div><span style="background-color: lightgrey"><font color="blue">this should also be blue over lightgrey</font></span></div></div> FYI, I can't reproduce this failure locally on my Snow Leopard machine. 586630 1 rniwa 2012-03-23 12:49:19 -0700 Oops, I menat Lion. 586651 2 rniwa 2012-03-23 13:04:11 -0700 Added a failing expectation in http://trac.webkit.org/changeset/111891. 586745 3 mitz 2012-03-23 14:23:16 -0700 Do you know when this started failing? 586759 4 rniwa 2012-03-23 14:31:45 -0700 (In reply to comment #3) > Do you know when this started failing? Unfortunately not. It's weird that this failure only occurs on Lion. 586761 5 mitz 2012-03-23 14:35:15 -0700 Lion is the first OS X release with autocorrect. Perhaps it’s kicking in during the test. I have not seen this failure earlier today when looking for “tests failing more than once on the Lion WebKit2 bots in a 40-50 revision range”, so I suspect it’s recent. Perhaps it’s a bot configuration issue involving autocorrect. 586779 6 rniwa 2012-03-23 14:54:35 -0700 (In reply to comment #5) > Lion is the first OS X release with autocorrect. Perhaps it’s kicking in during the test. I have not seen this failure earlier today when looking for “tests failing more than once on the Lion WebKit2 bots in a 40-50 revision range”, so I suspect it’s recent. Perhaps it’s a bot configuration issue involving autocorrect. Possible but I'm still surprised that the element moved from one place to another. Maybe auto-correction code is changing the selection in some undesirable fashion? 612838 7 139500 kojii 2012-04-30 12:03:02 -0700 Created attachment 139500 Patch 612891 8 rniwa 2012-04-30 13:12:41 -0700 Thanks for the fix! 613119 9 139500 kojii 2012-04-30 18:04:05 -0700 Comment on attachment 139500 Patch Thanks for the review! 613170 10 139500 webkit.review.bot 2012-04-30 19:08:24 -0700 Comment on attachment 139500 Patch Clearing flags on attachment: 139500 Committed r115700: <http://trac.webkit.org/changeset/115700> 613171 11 webkit.review.bot 2012-04-30 19:08:32 -0700 All reviewed patches have been landed. Closing bug. 139500 2012-04-30 12:03:02 -0700 2012-04-30 19:08:24 -0700 Patch bug-82079-20120501040259.patch text/plain 3264 kojii U3VidmVyc2lvbiBSZXZpc2lvbjogMTE1NjA2CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggNzk1NzQ1YjczMGMzMDE0YmMzMDY0 YjJlMzUzODUxYjI4ODliNzAwNS4uYzdiNGE1NmQ4Njc5N2FjZDMwMmVkN2VhYzQxNWJlMTZhYzk5 YWVjNCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEyLTA0LTMwICBLb2ppIElzaGlpICA8a29q aWlzaGlAZ21haWwuY29tPgorCisgICAgICAgIFJFR1JFU1NJT046IGVkaXRpbmcvaW5zZXJ0aW5n L2luc2VydC1iZy1mb250Lmh0bWwgZmFpbHMgb24gTGlvbgorICAgICAgICBodHRwczovL2J1Z3Mu d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODIwNzkKKworICAgICAgICBSZXZpZXdlZCBieSBO T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUaGUgdGVzdCBmYWlscyBvbmx5IGlmICJDb3JyZWN0 IHNwZWxsaW5nIGF1dG9tYXRpY2FsbHkiIGlzIHR1cm5lZCBvbi4KKyAgICAgICAgQXV0b0NvcnJl Y3Qga2lja3MgaW4gYmVjYXVzZSB0aGUgdGVzdCBoYXMgImJhY2tncm91bmRibHVlIiBhcyBhIHdv cmQsCisgICAgICAgIHNvIGNoYW5naW5nIGl0IHRvICJiYWNrZ3JvdW5kIGJsdWUiIGZpeGVzIHRo ZSBmYWlsdXJlLgorCisgICAgICAgICogZWRpdGluZy9pbnNlcnRpbmcvaW5zZXJ0LWJnLWZvbnQt ZXhwZWN0ZWQudHh0OiBDaGFuZ2UgImJhY2tncm91bmRibHVlIiB0byAiYmFja2dyb3VuZCBibHVl Ii4KKyAgICAgICAgKiBlZGl0aW5nL2luc2VydGluZy9pbnNlcnQtYmctZm9udC5odG1sOiBEaXR0 by4KKwogMjAxMi0wNC0zMCAgQW5kcmVhcyBLbGluZyAgPGtsaW5nQHdlYmtpdC5vcmc+CiAKICAg ICAgICAgQWRkIE1hYyBleHBlY3RhdGlvbiBmb3IgZmFzdC9kb20vV2luZG93L3dpbmRvdy1sb29r dXAtcHJlY2VkZW5jZS5odG1sLgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZWRpdGluZy9pbnNl cnRpbmcvaW5zZXJ0LWJnLWZvbnQtZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvZWRpdGluZy9p bnNlcnRpbmcvaW5zZXJ0LWJnLWZvbnQtZXhwZWN0ZWQudHh0CmluZGV4IDRmOWJhYzJjN2ExOWQw ZjllNWQ0NzNlODZhOTA4NjRiYWI0OWQzZTIuLmU4OGRiZTU3ZDIwMjY5YThiYzJmOGQyYmM2NTFj YmVmODhhNGQ4YmMgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL2VkaXRpbmcvaW5zZXJ0aW5nL2lu c2VydC1iZy1mb250LWV4cGVjdGVkLnR4dAorKysgYi9MYXlvdXRUZXN0cy9lZGl0aW5nL2luc2Vy dGluZy9pbnNlcnQtYmctZm9udC1leHBlY3RlZC50eHQKQEAgLTEsNSArMSw1IEBACiAKLSAgICAg ICAgPGRpdiBpZD0iZHYiIGNvbnRlbnRlZGl0YWJsZT0iIj48c3BhbiBzdHlsZT0iYmFja2dyb3Vu ZC1jb2xvcjogbGlnaHRncmV5Ij5saWdodGdyZXkgYmFja2dyb3VuZDxmb250IGNvbG9yPSJibHVl Ij5ibHVlIGZvbnQgY29sb3Igb3ZlciBsaWdodGdyZXkgYmFja2dyb3VuZDwvZm9udD48L3NwYW4+ IAorICAgICAgICA8ZGl2IGlkPSJkdiIgY29udGVudGVkaXRhYmxlPSIiPjxzcGFuIHN0eWxlPSJi YWNrZ3JvdW5kLWNvbG9yOiBsaWdodGdyZXkiPmxpZ2h0Z3JleSBiYWNrZ3JvdW5kIDxmb250IGNv bG9yPSJibHVlIj5ibHVlIGZvbnQgY29sb3Igb3ZlciBsaWdodGdyZXkgYmFja2dyb3VuZDwvZm9u dD48L3NwYW4+IAogPGRpdj48c3BhbiBzdHlsZT0iYmFja2dyb3VuZC1jb2xvcjogbGlnaHRncmV5 Ij48Zm9udCBjb2xvcj0iYmx1ZSI+dGhpcyBzaG91bGQgYWxzbyBiZSBibHVlIG92ZXIgbGlnaHRn cmV5PC9mb250Pjwvc3Bhbj48L2Rpdj48L2Rpdj4KICAgICAKIApkaWZmIC0tZ2l0IGEvTGF5b3V0 VGVzdHMvZWRpdGluZy9pbnNlcnRpbmcvaW5zZXJ0LWJnLWZvbnQuaHRtbCBiL0xheW91dFRlc3Rz L2VkaXRpbmcvaW5zZXJ0aW5nL2luc2VydC1iZy1mb250Lmh0bWwKaW5kZXggNWRkOGZmYWNlYmQy OTc4YTYwNmI1ZDRkNWQ2YjYxZDY4OWQ0NmQ0YS4uZTQ0ZmFmNTIzOTI5NTlhOGYxY2NhYTViNDA2 N2Q2N2QyZDM2ODk0MyAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvZWRpdGluZy9pbnNlcnRpbmcv aW5zZXJ0LWJnLWZvbnQuaHRtbAorKysgYi9MYXlvdXRUZXN0cy9lZGl0aW5nL2luc2VydGluZy9p bnNlcnQtYmctZm9udC5odG1sCkBAIC0xNSw3ICsxNSw3IEBACiAgICAgICAgIDwvc2NyaXB0Pgog ICAgIDwvaGVhZD4KICAgICA8Ym9keSBvbmxvYWQ9InRlc3QoKSI+Ci0gICAgICAgIDxkaXYgaWQ9 ImR2IiBjb250ZW50ZWRpdGFibGU+PHNwYW4gc3R5bGU9ImJhY2tncm91bmQtY29sb3I6IGxpZ2h0 Z3JleSI+bGlnaHRncmV5IGJhY2tncm91bmQ8Zm9udCBjb2xvcj0iYmx1ZSI+Ymx1ZSBmb250IGNv bG9yIG92ZXIgbGlnaHRncmV5IGJhY2tncm91bmQ8L2ZvbnQ+PC9zcGFuPiAKKyAgICAgICAgPGRp diBpZD0iZHYiIGNvbnRlbnRlZGl0YWJsZT48c3BhbiBzdHlsZT0iYmFja2dyb3VuZC1jb2xvcjog bGlnaHRncmV5Ij5saWdodGdyZXkgYmFja2dyb3VuZCA8Zm9udCBjb2xvcj0iYmx1ZSI+Ymx1ZSBm b250IGNvbG9yIG92ZXIgbGlnaHRncmV5IGJhY2tncm91bmQ8L2ZvbnQ+PC9zcGFuPiAKIDwvZGl2 PgogICAgIDwvYm9keT4KIDwvaHRtbD4KZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL3BsYXRmb3Jt L21hYy1saW9uL1NraXBwZWQgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtbGlvbi9Ta2lwcGVk CmluZGV4IDcwMmI3OGM4YjIwOWM2MjRiODMwYTFhZmFlODlkMzY2YzllZmY4NDAuLjIwOGE0YTg0 MDEzZDFjMzZiNzcyZGQxMWU3MTEyYWE0MzE5NmM4MzggMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3Rz L3BsYXRmb3JtL21hYy1saW9uL1NraXBwZWQKKysrIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFj LWxpb24vU2tpcHBlZApAQCAtMzUsOSArMzUsNiBAQCBlZGl0aW5nL3RleHQtaXRlcmF0b3IvZmlu ZFN0cmluZy1zZWxlY3Rpb24tZGlzYWJsZWQuaHRtbAogIyBQYXN0aW5nIHJlcGxhY2VzIHRoZSB0 aHJlZSBwZXJpb2RzIHdpdGggYW4gZWxsaXBzaXMgb24gTGlvbgogZWRpdGluZy9wYXN0ZWJvYXJk LzU0NzgyNTAuaHRtbAogCi0jIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD04MjA3OQotZWRpdGluZy9pbnNlcnRpbmcvaW5zZXJ0LWJnLWZvbnQuaHRtbAotCiAjIC0tLSBN YXRoIE1MIC0tLQogCiAjIFRoZXNlIGhhdmUgZGlmZmVyZW50IG1ldHJpY3MsIGFuZCByZXF1aXJl IGludmVzdGlnYXRpb24K