49879 2010-11-21 05:01:27 -0800 [Qt] FontPlatformDataQt: Use pixelsize not pointsize 2010-11-22 14:08:21 -0800 1 1 1 Unclassified WebKit WebKit Qt 528+ (Nightly build) PC OS X 10.5 RESOLVED INVALID Qt, QtTriaged P2 Normal --- 49759 1 robert webkit-unassigned benjamin oldest_to_newest 312069 0 robert 2010-11-21 05:01:27 -0800 QFont::pointSizeF always returns -1 if you use setPixelSize() like FontPlatformDataQt. So use pixelSize rather than pointSizeF elsewhere in the class. 312070 1 74499 robert 2010-11-21 05:04:02 -0800 Created attachment 74499 Patch 312075 2 robert 2010-11-21 05:37:06 -0800 Benjamin, does this look correct to you? 74499 2010-11-21 05:04:02 -0800 2010-11-22 14:08:21 -0800 Patch bug-49879-20101121130400.patch text/plain 2783 robert ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg YzExZTMyOTViYzU3NGJjNDQ1N2M3NGJiYzdkMDMwZDFkYWU1NmQxNy4uZjdhYWZlMjExYmE1NDNi OGUyYzA4NjQ0OTBjMWUwMWI4NWY1YjFjMyAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwyMCBAQAorMjAxMC0xMS0yMSAgUm9i ZXJ0IEhvZ2FuICA8cm9iZXJ0QHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C T0RZIChPT1BTISkuCisKKyAgICAgICAgW1F0XSBGb250UGxhdGZvcm1EYXRhUXQ6IFVzZSBwaXhl bHNpemUgbm90IHBvaW50c2l6ZQorCisgICAgICAgIFFGb250Ojpwb2ludFNpemVGIGFsd2F5cyBy ZXR1cm5zIC0xIGlmIHlvdSB1c2Ugc2V0UGl4ZWxTaXplKCkKKyAgICAgICAgbGlrZSBGb250UGxh dGZvcm1EYXRhUXQuIFNvIHVzZSBwaXhlbFNpemUgcmF0aGVyIHRoYW4KKyAgICAgICAgcG9pbnRT aXplRiBlbHNld2hlcmUgaW4gdGhlIGNsYXNzLgorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr aXQub3JnL3Nob3dfYnVnLmNnaT9pZD00OTg3OQorCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhp Y3MvcXQvRm9udFBsYXRmb3JtRGF0YS5oOgorICAgICAgICAoV2ViQ29yZTo6Rm9udFBsYXRmb3Jt RGF0YVByaXZhdGU6OkZvbnRQbGF0Zm9ybURhdGFQcml2YXRlKToKKyAgICAgICAgKiBwbGF0Zm9y bS9ncmFwaGljcy9xdC9Gb250UGxhdGZvcm1EYXRhUXQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6 Rm9udFBsYXRmb3JtRGF0YTo6Rm9udFBsYXRmb3JtRGF0YSk6CisKIDIwMTAtMTEtMjAgIEhlbGRl ciBDb3JyZWlhICA8aGVsZGVyQHNlbmNoYS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQW5k cmVhcyBLbGluZy4KZGlmZiAtLWdpdCBhL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvcXQvRm9u dFBsYXRmb3JtRGF0YS5oIGIvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9xdC9Gb250UGxhdGZv cm1EYXRhLmgKaW5kZXggMjIwMWYxODVkODE1OTUxOTIyNmUyMDQxYTA5NDFmYTBlYWZkNGE2YS4u N2FjYjQxNTQ5MWY0MzMwOGQzYjkyMzk5ZGI0ZmZiNTE4NGM4ZDcxZiAxMDA2NDQKLS0tIGEvV2Vi Q29yZS9wbGF0Zm9ybS9ncmFwaGljcy9xdC9Gb250UGxhdGZvcm1EYXRhLmgKKysrIGIvV2ViQ29y ZS9wbGF0Zm9ybS9ncmFwaGljcy9xdC9Gb250UGxhdGZvcm1EYXRhLmgKQEAgLTM2LDcgKzM2LDcg QEAgY2xhc3MgRm9udFBsYXRmb3JtRGF0YVByaXZhdGUgOiBwdWJsaWMgTm9uY29weWFibGUgewog cHVibGljOgogICAgIEZvbnRQbGF0Zm9ybURhdGFQcml2YXRlKCkKICAgICAgICAgOiByZWZDb3Vu dCgxKQotICAgICAgICAsIHNpemUoZm9udC5wb2ludFNpemVGKCkpCisgICAgICAgICwgc2l6ZShm b250LnBpeGVsU2l6ZSgpKQogICAgICAgICAsIGJvbGQoZm9udC5ib2xkKCkpCiAgICAgICAgICwg b2JsaXF1ZShmYWxzZSkKICAgICB7fQpAQCAtNDksNyArNDksNyBAQCBwdWJsaWM6CiAgICAgRm9u dFBsYXRmb3JtRGF0YVByaXZhdGUoY29uc3QgUUZvbnQmIGZvbnQpCiAgICAgICAgIDogcmVmQ291 bnQoMSkKICAgICAgICAgLCBmb250KGZvbnQpCi0gICAgICAgICwgc2l6ZShmb250LnBvaW50U2l6 ZUYoKSkKKyAgICAgICAgLCBzaXplKGZvbnQucGl4ZWxTaXplKCkpCiAgICAgICAgICwgYm9sZChm b250LmJvbGQoKSkKICAgICAgICAgLCBvYmxpcXVlKGZhbHNlKQogICAgIHt9CmRpZmYgLS1naXQg YS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3F0L0ZvbnRQbGF0Zm9ybURhdGFRdC5jcHAgYi9X ZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3F0L0ZvbnRQbGF0Zm9ybURhdGFRdC5jcHAKaW5kZXgg MDU2NWRlYmE3OTg0MGI1OWZlNTdmNGVkNzlhZjZhYjQ4OGNlMzc1NS4uMmUwZTlkZDk0NjdhZWFi YjQ4OTNmNTI1MDRmZmJmMGIzYTQ0MDJlNCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9wbGF0Zm9ybS9n cmFwaGljcy9xdC9Gb250UGxhdGZvcm1EYXRhUXQuY3BwCisrKyBiL1dlYkNvcmUvcGxhdGZvcm0v Z3JhcGhpY3MvcXQvRm9udFBsYXRmb3JtRGF0YVF0LmNwcApAQCAtNTEsNyArNTEsNyBAQCBGb250 UGxhdGZvcm1EYXRhOjpGb250UGxhdGZvcm1EYXRhKGNvbnN0IEZvbnREZXNjcmlwdGlvbiYgZGVz Y3JpcHRpb24sIGNvbnN0IEF0bwogewogICAgIFFGb250JiBmb250ID0gbV9kYXRhLT5mb250Owog ICAgIGZvbnQuc2V0RmFtaWx5KGZhbWlseU5hbWUpOwotICAgIGZvbnQuc2V0UGl4ZWxTaXplKHFS b3VuZChkZXNjcmlwdGlvbi5jb21wdXRlZFNpemUoKSkpOworICAgIGZvbnQuc2V0UGl4ZWxTaXpl KHFSb3VuZChkZXNjcmlwdGlvbi5jb21wdXRlZFBpeGVsU2l6ZSgpKSk7CiAgICAgZm9udC5zZXRJ dGFsaWMoZGVzY3JpcHRpb24uaXRhbGljKCkpOwogICAgIGZvbnQuc2V0V2VpZ2h0KHRvUUZvbnRX ZWlnaHQoZGVzY3JpcHRpb24ud2VpZ2h0KCkpKTsKICAgICBmb250LnNldFdvcmRTcGFjaW5nKHdv cmRTcGFjaW5nKTsKQEAgLTYzLDcgKzYzLDcgQEAgRm9udFBsYXRmb3JtRGF0YTo6Rm9udFBsYXRm b3JtRGF0YShjb25zdCBGb250RGVzY3JpcHRpb24mIGRlc2NyaXB0aW9uLCBjb25zdCBBdG8KICNl bmRpZgogCiAgICAgbV9kYXRhLT5ib2xkID0gZm9udC5ib2xkKCk7Ci0gICAgbV9kYXRhLT5zaXpl ID0gZm9udC5wb2ludFNpemVGKCk7CisgICAgbV9kYXRhLT5zaXplID0gZm9udC5waXhlbFNpemUo KTsKIH0KIAogRm9udFBsYXRmb3JtRGF0YTo6fkZvbnRQbGF0Zm9ybURhdGEoKQo=