104908 2012-12-13 06:16:48 -0800 [Qt4] Rendering issue when using CSS ellipsis text-overflow 2012-12-13 07:13:05 -0800 1 1 1 Unclassified WebKit Text 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 88186 0 jbriance webkit-unassigned allan.jensen oldest_to_newest 790541 0 179260 jbriance 2012-12-13 06:16:48 -0800 Created attachment 179260 Patch for qtwebkit 2.3 Offset computation in offsetForPositionForSimpleText is not correct in Source/WebCore/platform/graphics/qt/FontQt4.cpp implementation whereas the equivalent offset computation used in Qt5 in Source/WebCore/platform/graphics/FontFastPath.cpp is correct. You can compare the result of webkit test LayoutTests/fast/css/text-overflow-ellipsis.html between Qt 4 and Qt 5 to stimulate this issue. 790584 1 allan.jensen 2012-12-13 07:13:05 -0800 Landed in qtwebkit-2.3-staging by commit ec61316964f0d7088f9532c3edca9e468bf870c7 179260 2012-12-13 06:16:48 -0800 2012-12-13 06:16:48 -0800 Patch for qtwebkit 2.3 0001-Qt4-Fix-rendering-issue-when-using-CSS-ellipsis-text.patch text/plain 1500 jbriance RnJvbSBkOWMxZTYxZjJmZTFiMGFjM2U2YTVhMTlhZmU4ZGY3ZWU3MjJlYzU1IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBKdWxpZW4gQnJpYW5jZWF1IDxqYnJpYW5jZWF1QG5kcy5jb20+ CkRhdGU6IFRodSwgMTMgRGVjIDIwMTIgMTU6MTI6MTIgKzAxMDAKU3ViamVjdDogW1BBVENIXSBb UXQ0XSBGaXggcmVuZGVyaW5nIGlzc3VlIHdoZW4gdXNpbmcgQ1NTIGVsbGlwc2lzCiB0ZXh0LW92 ZXJmbG93CgpPZmZzZXQgY29tcHV0YXRpb24gaW4gb2Zmc2V0Rm9yUG9zaXRpb25Gb3JTaW1wbGVU ZXh0IGlzIG5vdCBjb3JyZWN0IGluIEZvbnRRdDQuY3BwIGltcGxlbWVudGF0aW9uLgpXZWJLaXQg dGVzdCBMYXlvdXRUZXN0cy9mYXN0L2Nzcy90ZXh0LW92ZXJmbG93LWVsbGlwc2lzLmh0bWwgY2Fu IGJlIHVzZWQgdG8gcmVwcm9kdWNlIHRoaXMgaXNzdWUuCi0tLQogU291cmNlL1dlYkNvcmUvcGxh dGZvcm0vZ3JhcGhpY3MvcXQvRm9udFF0NC5jcHAgfCA0ICsrLS0KIDEgZmlsZSBjaGFuZ2VkLCAy IGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNv cmUvcGxhdGZvcm0vZ3JhcGhpY3MvcXQvRm9udFF0NC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0 Zm9ybS9ncmFwaGljcy9xdC9Gb250UXQ0LmNwcAppbmRleCAwNTliYjkyLi44M2JmYWU3IDEwMDY0 NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9xdC9Gb250UXQ0LmNwcAor KysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9xdC9Gb250UXQ0LmNwcApAQCAt MzI3LDcgKzMyNyw3IEBAIGludCBGb250OjpvZmZzZXRGb3JQb3NpdGlvbkZvclNpbXBsZVRleHQo Y29uc3QgVGV4dFJ1biYgcnVuLCBmbG9hdCBwb3NpdGlvbiwgYm9vCiAgICAgU3RyaW5nIHNhbml0 aXplZCA9IEZvbnQ6Om5vcm1hbGl6ZVNwYWNlcyhydW4uY2hhcmFjdGVyczE2KCksIHJ1bi5sZW5n dGgoKSk7CiAgICAgUVN0cmluZyBzdHJpbmcgPSBmcm9tUmF3RGF0YVdpdGhvdXRSZWYoc2FuaXRp emVkKTsKIAotICAgIFFGb250TWV0cmljcyBmbShmb250KCkpOworICAgIFFGb250TWV0cmljc0Yg Zm0oZm9udCgpKTsKICAgICBmbG9hdCBkZWx0YSA9IHBvc2l0aW9uOwogICAgIGludCBjdXJQb3Mg PSAwOwogICAgIGRvIHsKQEAgLTMzNyw3ICszMzcsNyBAQCBpbnQgRm9udDo6b2Zmc2V0Rm9yUG9z aXRpb25Gb3JTaW1wbGVUZXh0KGNvbnN0IFRleHRSdW4mIHJ1biwgZmxvYXQgcG9zaXRpb24sIGJv bwogICAgICAgICAgICAgaWYgKGRlbHRhICsgY2hhcldpZHRoIC8gMiA8PSAwKQogICAgICAgICAg ICAgICAgIGJyZWFrOwogICAgICAgICB9IGVsc2UgewotICAgICAgICAgICAgaWYgKGRlbHRhICsg Y2hhcldpZHRoIDw9IDApCisgICAgICAgICAgICBpZiAoZGVsdGEgPD0gMCkKICAgICAgICAgICAg ICAgICBicmVhazsKICAgICAgICAgfQogICAgIH0gd2hpbGUgKCsrY3VyUG9zIDwgc3RyaW5nLnNp emUoKSk7Ci0tIAoxLjguMAoK