104391 2012-12-07 12:40:22 -0800 [BlackBerry] Strip whitespace from learned text 2012-12-07 14:05:25 -0800 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 nima.ghanavatian nima.ghanavatian mifenton rwlbuis tonikitoo webkit.review.bot yong.li.webkit oldest_to_newest 786062 0 nima.ghanavatian 2012-12-07 12:40:22 -0800 Instead of using remove(), we should be use stripWhiteSpace here, to not only correctly remove whitespace at the start of text but also at the end. 786065 1 178258 nima.ghanavatian 2012-12-07 12:42:04 -0800 Created attachment 178258 Patch 786069 2 178258 rwlbuis 2012-12-07 12:44:09 -0800 Comment on attachment 178258 Patch LGTM. 786127 3 178258 webkit.review.bot 2012-12-07 14:05:21 -0800 Comment on attachment 178258 Patch Clearing flags on attachment: 178258 Committed r136981: <http://trac.webkit.org/changeset/136981> 786128 4 webkit.review.bot 2012-12-07 14:05:25 -0800 All reviewed patches have been landed. Closing bug. 178258 2012-12-07 12:42:04 -0800 2012-12-07 14:05:21 -0800 Patch bug-104391-20121207153358.patch text/plain 1755 nima.ghanavatian U3VidmVyc2lvbiBSZXZpc2lvbjogMTM2OTYxCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2Js YWNrYmVycnkvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0NoYW5nZUxvZwpp bmRleCAxMzVjNzI2Nzk1YzQ5MjBlOWZkMzBkMDRkNzdlYTIzMmQxNGE1NWNkLi40YzhjNzU0ZTYw YjUzYTFhZGRlMTQ1NGU3NTUzODlmOWNjODdkYzhmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 L2JsYWNrYmVycnkvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFu Z2VMb2cKQEAgLTEsMyArMSwxOCBAQAorMjAxMi0xMi0wNSAgTmltYSBHaGFuYXZhdGlhbiAgPG5n aGFuYXZhdGlhbkByaW0uY29tPgorCisgICAgICAgIFtCbGFja0JlcnJ5XSBTdHJpcCB3aGl0ZXNw YWNlIGZyb20gbGVhcm5lZCB0ZXh0LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z aG93X2J1Zy5jZ2k/aWQ9MTA0MzkxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT ISkuCisKKyAgICAgICAgVGhpcyB3YXMgcmVtb3ZpbmcgdGhlIGZpcnN0IHdvcmQgZnJvbSB0aGUg c2VudGVuY2UgYmVpbmcgbGVhcm5lZC4gVXNpbmcKKyAgICAgICAgc3RyaXBXaGl0ZVNwYWNlIHRv IHJlbW92ZSBzcGFjZXMgZnJvbSB0aGUgc3RhcnQgYW5kIGVuZCBvZiB0aGUgdGV4dC4KKworICAg ICAgICBJbnRlcm5hbGx5IHJldmlld2VkIGJ5IE1pa2UgRmVudG9uLgorCisgICAgICAgICogV2Vi S2l0U3VwcG9ydC9JbnB1dEhhbmRsZXIuY3BwOgorICAgICAgICAoQmxhY2tCZXJyeTo6V2ViS2l0 OjpJbnB1dEhhbmRsZXI6OmxlYXJuVGV4dCk6CisKIDIwMTItMTItMDcgIEFuZHJldyBMbyAgPGFu bG9AcmltLmNvbT4KIAogICAgICAgICBbQmxhY2tCZXJyeV0gRG91YmxlIHpvb20gYW5pbWF0aW9u IHdoZW4gZm9jdXNpbmcgb24gaW5wdXQgZmllbGQuCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0 L2JsYWNrYmVycnkvV2ViS2l0U3VwcG9ydC9JbnB1dEhhbmRsZXIuY3BwIGIvU291cmNlL1dlYktp dC9ibGFja2JlcnJ5L1dlYktpdFN1cHBvcnQvSW5wdXRIYW5kbGVyLmNwcAppbmRleCAyOWRiY2Jh NzQ3ODEwMGUxMzcxNGU2NDdhNjg3OTM5ZDhlYWRiZTJiLi41ZTBmOTEzN2FhNjVkOTkyM2EyMTAy MDlkZGE1YTgxYThiNWM5YTBjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkv V2ViS2l0U3VwcG9ydC9JbnB1dEhhbmRsZXIuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2ti ZXJyeS9XZWJLaXRTdXBwb3J0L0lucHV0SGFuZGxlci5jcHAKQEAgLTU2Nyw3ICs1NjcsNyBAQCB2 b2lkIElucHV0SGFuZGxlcjo6bGVhcm5UZXh0KCkKIAogICAgIFdURjo6U3RyaW5nIHRleHRJbkZp ZWxkKGVsZW1lbnRUZXh0KCkpOwogICAgIHRleHRJbkZpZWxkID0gdGV4dEluRmllbGQuc3Vic3Ry aW5nKHN0ZDo6bWF4KDAsIHN0YXRpY19jYXN0PGludD4odGV4dEluRmllbGQubGVuZ3RoKCkgLSBN YXhMZWFyblRleHREYXRhU2l6ZSkpLCB0ZXh0SW5GaWVsZC5sZW5ndGgoKSk7Ci0gICAgdGV4dElu RmllbGQucmVtb3ZlKDAsIHRleHRJbkZpZWxkLmZpbmQoIiAiKSk7CisgICAgdGV4dEluRmllbGQg PSB0ZXh0SW5GaWVsZC5zdHJpcFdoaXRlU3BhY2UoKTsKIAogICAgIC8vIEJ1aWxkIHVwIHRoZSA1 MDAgY2hhcmFjdGVyIHN0cmluZ3MgaW4gd29yZCBjaHVua3MuCiAgICAgLy8gU3BlYyBzYXlzIDEw MDAsIGJ1dCBtZW1vcnkgY29ycnVwdGlvbiBoYXMgYmVlbiBvYnNlcnZlZC4K