82198 2012-03-26 06:36:38 -0700 Insertion point jumps when using link detection 2013-04-29 15:24:14 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) Mac OS X 10.7 RESOLVED FIXED P2 Normal --- 85463 1 dev-bugzilla webkit-unassigned adele ap commit-queue enrica jiapu.mail max.hong.shen mrowe rniwa oldest_to_newest 587667 0 dev-bugzilla 2012-03-26 06:36:38 -0700 Steps to reproduce: 1. Open the attached HTML document in Safari. 2. Make sure automatic link detection is enabled (Edit > Substitutions > Smart Links). 3. Place the insertion point at the beginning of the text "Lorem ipsum". 4. Type "www.equinux.com". Expected Results: Text field should contain this: www.equinux.comLorem ipsum Actual Results: After typing the second period the insertion point jumps past the word "Lorem". In the end, the text field contains this: www.equinux.Lorem comipsum 587668 1 133792 dev-bugzilla 2012-03-26 06:37:11 -0700 Created attachment 133792 Sample document 615028 2 rniwa 2012-05-02 23:08:38 -0700 Could someone triage this bug? 615062 3 ap 2012-05-03 00:00:44 -0700 I can reproduce this in Safari 5.1.5, but in ToT, smart links appear to be completely broken. Filed bug 85463. 881620 4 199766 max.hong.shen 2013-04-25 17:08:37 -0700 Created attachment 199766 proposed patch 881629 5 rniwa 2013-04-25 17:38:36 -0700 I can’t reproduce this problem anymore. Can you? 882762 6 max.hong.shen 2013-04-29 14:28:16 -0700 Thanks for review. Yes, I can reproduce this bug on Mac with the latest webkit code. (In reply to comment #5) > I can’t reproduce this problem anymore. Can you? 882790 7 199766 commit-queue 2013-04-29 15:24:12 -0700 Comment on attachment 199766 proposed patch Clearing flags on attachment: 199766 Committed r149322: <http://trac.webkit.org/changeset/149322> 882791 8 commit-queue 2013-04-29 15:24:14 -0700 All reviewed patches have been landed. Closing bug. 133792 2012-03-26 06:37:11 -0700 2012-03-26 06:37:11 -0700 Sample document sample.html text/html 198 dev-bugzilla PGh0bWw+PGJvZHk+CgoJPGRpdiBjb250ZW50ZWRpdGFibGU9InRydWUiIHN0eWxlPSJmb250LXNp emU6IDE2cHg7IGZvbnQtZmFtaWx5OiAnTHVjaWRhIEdyYW5kZSc7ICI+PGZvbnQgZmFjZT0iQXJp YWwiIHN0eWxlPSJmb250LXNpemU6MTZweDsgbGluZS1oZWlnaHQ6MS41Ij5Mb3JlbSBpcHN1bTwv Zm9udD48L2Rpdj4KCjwvYm9keT48L2h0bWw+ 199766 2013-04-25 17:08:37 -0700 2013-04-29 15:24:11 -0700 proposed patch 82198v1.patch text/plain 6570 max.hong.shen ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCA1ODFiMmE0Li43OTdkZjA0IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTMtMDQt MjUgIFlpIFNoZW4gIDxtYXguaG9uZy5zaGVuQGdtYWlsLmNvbT4KKworICAgICAgICBIVE1MIEVk aXRpbmc6IEluc2VydGlvbiBwb2ludCBqdW1wcyB3aGVuIHVzaW5nIGxpbmsgZGV0ZWN0aW9uCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MjE5OAorCisg ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorICAgICAgICAKKyAgICAgICAgVXBk YXRlIHR5cGluZy1zcGFjZS10by10cmlnZ2VyLXNtYXJ0LWxpbmsuaHRtbCB0byB2ZXJpZnkgbGlu ayBkZXRlY3Rpb24gYmVoYXZpb3IKKyAgICAgICAgd2hlbiB0eXBpbmcgYSBsaW5rIGJlZm9yZSBl eGlzdGluZyB0ZXh0LgorCisgICAgICAgICogZWRpdGluZy9pbnNlcnRpbmcvdHlwaW5nLXNwYWNl LXRvLXRyaWdnZXItc21hcnQtbGluay1leHBlY3RlZC50eHQ6CisgICAgICAgICogZWRpdGluZy9p bnNlcnRpbmcvdHlwaW5nLXNwYWNlLXRvLXRyaWdnZXItc21hcnQtbGluay5odG1sOgorCiAyMDEz LTA0LTI1ICBSb21hbiBaaHV5a292ICA8emhyb21hQGlzcHJhcy5ydT4KIAogICAgICAgICBBcml0 aG1ldGljIG9wZXJhdGlvbnMgd2l0aCBuZWdhdGl2ZSB6ZXJvIHNob3VsZCBiZSBvcHRpbWl6ZWQg Y29ycmVjbHR5LgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZWRpdGluZy9pbnNlcnRpbmcvdHlw aW5nLXNwYWNlLXRvLXRyaWdnZXItc21hcnQtbGluay1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0 cy9lZGl0aW5nL2luc2VydGluZy90eXBpbmctc3BhY2UtdG8tdHJpZ2dlci1zbWFydC1saW5rLWV4 cGVjdGVkLnR4dAppbmRleCBhYTBkMGVlLi42MTYzZDZhIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0 cy9lZGl0aW5nL2luc2VydGluZy90eXBpbmctc3BhY2UtdG8tdHJpZ2dlci1zbWFydC1saW5rLWV4 cGVjdGVkLnR4dAorKysgYi9MYXlvdXRUZXN0cy9lZGl0aW5nL2luc2VydGluZy90eXBpbmctc3Bh Y2UtdG8tdHJpZ2dlci1zbWFydC1saW5rLWV4cGVjdGVkLnR4dApAQCAtMSwyICsxLDQgQEAKIFRo ZSB3d3cuZm9vLmNvbSBzaG91bGQgYmUgdW5kZXJsaW5lZCBhbmQgdGhlcmUgaXMgYW4gYW5jaG9y IG5vZGUgY3JlYXRlZCBmb3IgaXQuCitUaGUgd3d3LmJhci5jb20gc2hvdWxkIGJlIHVuZGVybGlu ZWQgYW5kIHRoZXJlIGlzIGFuIGFuY2hvciBub2RlIGNyZWF0ZWQgZm9yIGl0LgogUEFTUzogdGhl IGFuY2hvciBmb3IgJ3d3dy5mb28uY29tJyBoYXMgYmVlbiBjcmVhdGVkLgorUEFTUzogdGhlIGFu Y2hvciBmb3IgJ3d3dy5iYXIuY29tJyBoYXMgYmVlbiBjcmVhdGVkLgpkaWZmIC0tZ2l0IGEvTGF5 b3V0VGVzdHMvZWRpdGluZy9pbnNlcnRpbmcvdHlwaW5nLXNwYWNlLXRvLXRyaWdnZXItc21hcnQt bGluay5odG1sIGIvTGF5b3V0VGVzdHMvZWRpdGluZy9pbnNlcnRpbmcvdHlwaW5nLXNwYWNlLXRv LXRyaWdnZXItc21hcnQtbGluay5odG1sCmluZGV4IDRkNTliOTYuLjQzZDU3MDcgMTAwNjQ0Ci0t LSBhL0xheW91dFRlc3RzL2VkaXRpbmcvaW5zZXJ0aW5nL3R5cGluZy1zcGFjZS10by10cmlnZ2Vy LXNtYXJ0LWxpbmsuaHRtbAorKysgYi9MYXlvdXRUZXN0cy9lZGl0aW5nL2luc2VydGluZy90eXBp bmctc3BhY2UtdG8tdHJpZ2dlci1zbWFydC1saW5rLmh0bWwKQEAgLTksMTYgKzksMzYgQEAgZnVu Y3Rpb24gcnVuVGVzdCgpCiAgICAgICAgIHRlc3RSdW5uZXIuc2V0QXV0b21hdGljTGlua0RldGVj dGlvbkVuYWJsZWQodHJ1ZSk7CiAgICAgfQogCi0gICAgdmFyIHRhcmdldERpdiA9IGRvY3VtZW50 LmdldEVsZW1lbnRCeUlkKCd0ZXN0Jyk7Ci0gICAgdmFyIHRhcmdldFRleHQgPSB0YXJnZXREaXYu Zmlyc3RDaGlsZDsKKyAgICB2YXIgdGVzdFR5cGVTcGFjZURpdiA9IGRvY3VtZW50LmdldEVsZW1l bnRCeUlkKCd0ZXN0VHlwZVNwYWNlJyk7CisgICAgdmFyIHRhcmdldFRleHQgPSB0ZXN0VHlwZVNw YWNlRGl2LmZpcnN0Q2hpbGQ7CiAgICAgd2luZG93LmdldFNlbGVjdGlvbigpLnNldFBvc2l0aW9u KHRhcmdldFRleHQsIDE1KTsKICAgICBwcmVzc0tleSgiICIpOwotICAgIGRvY3VtZW50LmdldEVs ZW1lbnRCeUlkKCdsb2cnKS50ZXh0Q29udGVudCA9IHRhcmdldERpdi5pbm5lckhUTUw7CiAgICAg dmFyIGV4cGVjdGVkQ29udGVudHMgPSAiVGhlIDxhIGhyZWY9XCJodHRwOi8vd3d3LmZvby5jb21c Ij53d3cuZm9vLmNvbTwvYT4gc2hvdWxkIGJlIHVuZGVybGluZWQgYW5kIHRoZXJlIGlzIGFuIGFu Y2hvciBub2RlIGNyZWF0ZWQgZm9yIGl0LiI7Ci0gICAgaWYgKGV4cGVjdGVkQ29udGVudHMgPT0g dGFyZ2V0RGl2LmlubmVySFRNTCkKLSAgICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2xv ZycpLnRleHRDb250ZW50ID0gIlBBU1M6IHRoZSBhbmNob3IgZm9yICd3d3cuZm9vLmNvbScgaGFz IGJlZW4gY3JlYXRlZC4iCisgICAgaWYgKGV4cGVjdGVkQ29udGVudHMgPT0gdGVzdFR5cGVTcGFj ZURpdi5pbm5lckhUTUwpCisgICAgICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdsb2cnKS50 ZXh0Q29udGVudCA9ICJQQVNTOiB0aGUgYW5jaG9yIGZvciAnd3d3LmZvby5jb20nIGhhcyBiZWVu IGNyZWF0ZWQuXG4iCiAgICAgZWxzZQotICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgn bG9nJykudGV4dENvbnRlbnQgPSAiRmFpbGVkOiB0aGUgZXhwZWN0ZWQgY29udGVudCB3YXMgJyIg KyBleHBlY3RlZENvbnRlbnRzICsgIicsIGJ1dCB0aGUgYWN0dWFsIHJlc3VsdCB3YXMgJyIgKyB0 YXJnZXREaXYuaW5uZXJIVE1MICsgIicuIgorICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJ ZCgnbG9nJykudGV4dENvbnRlbnQgPSAiRmFpbGVkOiB0aGUgZXhwZWN0ZWQgY29udGVudCB3YXMg JyIgKyBleHBlY3RlZENvbnRlbnRzICsgIicsIGJ1dCB0aGUgYWN0dWFsIHJlc3VsdCB3YXMgJyIg KyB0ZXN0VHlwZVNwYWNlRGl2LmlubmVySFRNTCArICInLlxuIgorICAgIAorICAgIHZhciB0ZXN0 VHlwZUxpbmtEaXYgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgndGVzdFR5cGVMaW5rJyk7Cisg ICAgdGFyZ2V0VGV4dCA9IHRlc3RUeXBlTGlua0Rpdi5maXJzdENoaWxkOworICAgIHdpbmRvdy5n ZXRTZWxlY3Rpb24oKS5zZXRQb3NpdGlvbih0YXJnZXRUZXh0LCA0KTsKKyAgICBwcmVzc0tleSgi dyIpOworICAgIHByZXNzS2V5KCJ3Iik7CisgICAgcHJlc3NLZXkoInciKTsKKyAgICBwcmVzc0tl eSgiLiIpOworICAgIHByZXNzS2V5KCJiIik7CisgICAgcHJlc3NLZXkoImEiKTsKKyAgICBwcmVz c0tleSgiciIpOworICAgIHByZXNzS2V5KCIuIik7CisgICAgcHJlc3NLZXkoImMiKTsKKyAgICBw cmVzc0tleSgibyIpOworICAgIHByZXNzS2V5KCJtIik7CisgICAgcHJlc3NLZXkoIiAiKTsKKyAg ICBleHBlY3RlZENvbnRlbnRzID0gIlRoZSA8YSBocmVmPVwiaHR0cDovL3d3dy5iYXIuY29tXCI+ d3d3LmJhci5jb208L2E+IHNob3VsZCBiZSB1bmRlcmxpbmVkIGFuZCB0aGVyZSBpcyBhbiBhbmNo b3Igbm9kZSBjcmVhdGVkIGZvciBpdC4iOworICAgIGlmIChleHBlY3RlZENvbnRlbnRzID09IHRl c3RUeXBlTGlua0Rpdi5pbm5lckhUTUwpCisgICAgICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlk KCdsb2cnKS50ZXh0Q29udGVudCArPSAiUEFTUzogdGhlIGFuY2hvciBmb3IgJ3d3dy5iYXIuY29t JyBoYXMgYmVlbiBjcmVhdGVkLiIKKyAgICBlbHNlCisgICAgICAgIGRvY3VtZW50LmdldEVsZW1l bnRCeUlkKCdsb2cnKS50ZXh0Q29udGVudCArPSAiRmFpbGVkOiB0aGUgZXhwZWN0ZWQgY29udGVu dCB3YXMgJyIgKyBleHBlY3RlZENvbnRlbnRzICsgIicsIGJ1dCB0aGUgYWN0dWFsIHJlc3VsdCB3 YXMgJyIgKyB0ZXN0VHlwZUxpbmtEaXYuaW5uZXJIVE1MICsgIicuIgogCiAgICAgaWYgKHdpbmRv dy50ZXN0UnVubmVyKQogICAgICAgICB0ZXN0UnVubmVyLnNldEF1dG9tYXRpY0xpbmtEZXRlY3Rp b25FbmFibGVkKGZhbHNlKTsKQEAgLTM5LDcgKzU5LDggQEAgZnVuY3Rpb24gcHJlc3NLZXkoa2V5 KQogPC9zY3JpcHQ+CiA8L2hlYWQ+CiA8Ym9keT4KLTxkaXYgaWQ9InRlc3QiIGNvbnRlbnRlZGl0 YWJsZT5UaGUgd3d3LmZvby5jb21zaG91bGQgYmUgdW5kZXJsaW5lZCBhbmQgdGhlcmUgaXMgYW4g YW5jaG9yIG5vZGUgY3JlYXRlZCBmb3IgaXQuPC9kaXY+Cis8ZGl2IGlkPSJ0ZXN0VHlwZVNwYWNl IiBjb250ZW50ZWRpdGFibGU+VGhlIHd3dy5mb28uY29tc2hvdWxkIGJlIHVuZGVybGluZWQgYW5k IHRoZXJlIGlzIGFuIGFuY2hvciBub2RlIGNyZWF0ZWQgZm9yIGl0LjwvZGl2PgorPGRpdiBpZD0i dGVzdFR5cGVMaW5rIiBjb250ZW50ZWRpdGFibGU+VGhlIHNob3VsZCBiZSB1bmRlcmxpbmVkIGFu ZCB0aGVyZSBpcyBhbiBhbmNob3Igbm9kZSBjcmVhdGVkIGZvciBpdC48L2Rpdj4KIDxwcmUgaWQ9 ImxvZyI+PC9wcmU+CiA8L2JvZHk+CiA8c2NyaXB0PgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNv cmUvQ2hhbmdlTG9nIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4IDgxYjIzMzAuLmVh ZGQ5OWQgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9Tb3VyY2Uv V2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwyMSBAQAorMjAxMy0wNC0yNSAgWWkgU2hlbiAg PG1heC5ob25nLnNoZW5AZ21haWwuY29tPgorCisgICAgICAgIEhUTUwgRWRpdGluZzogSW5zZXJ0 aW9uIHBvaW50IGp1bXBzIHdoZW4gdXNpbmcgbGluayBkZXRlY3Rpb24KKyAgICAgICAgaHR0cHM6 Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTgyMTk4CisKKyAgICAgICAgUmV2aWV3 ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQWRkIGxpbmtzIHNob3VsZCBiZSBkb25l IG9ubHkgd2hlbiB0eXBpbmcgYSB3b3JkIHNlcGFyYXRvciAoZS5nLiBzcGFjZSBvciBkb3QpIHdo aWNoIAorICAgICAgICBicmVha3MgdGhlIHRleHQgaW50byB0d28gcGFydHMgLSB0aGUgbGluayBh bmQgdGhlIHJlc3Qgb2YgdGhlIHRleHQuIFJlZmxlY3RlZCBpbiAKKyAgICAgICAgY29kZSwgd2Ug c2hvdWxkIG9ubHkgYWRkIGxpbmsgd2hlbiB0aGUgY2FyZXQgcG9zaXRpb24gaXMgYXQgdGhlIHJp Z2h0IGVuZCBsb2NhdGlvbiBvZiAKKyAgICAgICAgVGV4dENoZWNraW5nIHJlc3VsdC4gT3RoZXJ3 aXNlLCB0aGUgdGV4dCBvbiB0aGUgcmlnaHQgb2YgYSB3b3JkIHNlcGFyYXRvciBjb3VsZCBiZQor ICAgICAgICBoaWdobGlnaHRlZCBhcyBhIGxpbmsgYW5kIHRoZSBpbnNlcnRpb24gcG9pbnQganVt cHMgdW5leHBlY3RlZGx5LgorCisgICAgICAgIFRlc3RzOiB1cGRhdGUgZWRpdGluZy9pbnNlcnRp bmcvdHlwaW5nLXNwYWNlLXRvLXRyaWdnZXItc21hcnQtbGluay5odG1sCisKKyAgICAgICAgKiBl ZGl0aW5nL0VkaXRvci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpFZGl0b3I6Om1hcmtBbmRSZXBs YWNlRm9yKToKKwogMjAxMy0wNC0yNSAgUWlhbmt1biBNaWFvICA8cWlhbmt1bi5taWFvQGludGVs LmNvbT4KIAogICAgICAgICBbQ1NTIFNoYWRlcnNdIFJlbW92ZSB0aGUgbWVzaFR5cGUgZnJvbSB0 aGUgQ3VzdG9tRmlsdGVyT3BlcmF0aW9uCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9lZGl0 aW5nL0VkaXRvci5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9lZGl0aW5nL0VkaXRvci5jcHAKaW5kZXgg Y2ZiMzcyNC4uMmI1ODI0YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvZWRpdGluZy9FZGl0 b3IuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL2VkaXRpbmcvRWRpdG9yLmNwcApAQCAtMjI1Nyw3 ICsyMjU3LDcgQEAgdm9pZCBFZGl0b3I6Om1hcmtBbmRSZXBsYWNlRm9yKFBhc3NSZWZQdHI8U3Bl bGxDaGVja1JlcXVlc3Q+IHJlcXVlc3QsIGNvbnN0IFZlY3QKICAgICAgICAgICAgIFJlZlB0cjxS YW5nZT4gcmFuZ2VUb1JlcGxhY2UgPSBwYXJhZ3JhcGguc3VicmFuZ2UocmVzdWx0TG9jYXRpb24s IHJlc3VsdExlbmd0aCk7CiAKICAgICAgICAgICAgIC8vIGFkZGluZyBsaW5rcyBzaG91bGQgYmUg ZG9uZSBvbmx5IGltbWVkaWF0ZWx5IGFmdGVyIHRoZXkgYXJlIHR5cGVkCi0gICAgICAgICAgICBp ZiAocmVzdWx0VHlwZSA9PSBUZXh0Q2hlY2tpbmdUeXBlTGluayAmJiAoc2VsZWN0aW9uT2Zmc2V0 ID4gcmVzdWx0RW5kTG9jYXRpb24gKyAxIHx8IHNlbGVjdGlvbk9mZnNldCA8PSByZXN1bHRMb2Nh dGlvbikpCisgICAgICAgICAgICBpZiAocmVzdWx0VHlwZSA9PSBUZXh0Q2hlY2tpbmdUeXBlTGlu ayAmJiBzZWxlY3Rpb25PZmZzZXQgIT0gcmVzdWx0RW5kTG9jYXRpb24gKyAxKQogICAgICAgICAg ICAgICAgIGNvbnRpbnVlOwogCiAgICAgICAgICAgICBpZiAoIShzaG91bGRQZXJmb3JtUmVwbGFj ZW1lbnQgfHwgc2hvdWxkQ2hlY2tGb3JDb3JyZWN0aW9uIHx8IHNob3VsZE1hcmtMaW5rKSB8fCAh ZG9SZXBsYWNlbWVudCkK