101443 2012-11-07 03:24:37 -0800 REGRESSION (r125578): word-wrapping in absolute position with nbsp, word-spacing and custom font 2019-01-09 05:26:15 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Mac (Intel) OS X 10.8 RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=193279 http://jsfiddle.net/5Htwx/1/ Regression P1 Normal --- 1 openmail+webkit robert ap eric glenn mitz openmail+webkit robert webkit.review.bot oldest_to_newest 760499 0 openmail+webkit 2012-11-07 03:24:37 -0800 In the example the div should only be one line of text, instead the line is wrapped. Broken in WebKit Nightly 8536.26.17, 537+ and Chrome 23.0.1271.64 Works in Safari 6.0.2, Firefox 16.0 Mac, Opera 12.02 Mac, Chrome 22 Mac 760500 1 172750 openmail+webkit 2012-11-07 03:25:29 -0800 Created attachment 172750 Example of the issue. 760804 2 mitz 2012-11-07 09:57:37 -0800 Also happens without the custom font. Probably a duplicate of bug 96856. 760809 3 ap 2012-11-07 09:59:32 -0800 This seems to have been an intentional change, see <http://trac.webkit.org/changeset/125578>. However, bug 89826 does not mention any testing performed, so perhaps CSS spec is just wrong. Why do you say that the new behavior is incorrect? Does the change break any live sites? 760823 4 mitz 2012-11-07 10:11:41 -0800 (In reply to comment #3) > This seems to have been an intentional change, see <http://trac.webkit.org/changeset/125578>. However, bug 89826 does not mention any testing performed, so perhaps CSS spec is just wrong. > > Why do you say that the new behavior is incorrect? Does the change break any live sites? This is incorrect because the box should stretch to the maximum preferred width of its contents (regardless of how word-spacing factors into that width), but instead it falls short, so the text is forced to wrap. When the behavior of word-spacing was changed in r125578, supposedly for compliance with the spec, the maximum width calculation in RenderText was not updated, causing a discrepancy that leads to this bug. 760825 5 openmail+webkit 2012-11-07 10:13:58 -0800 > Also happens without the custom font. You are right, I probably messed something up, custom font is not necessary to reproduce. > Why do you say that the new behavior is incorrect? Does the change break any live sites? Well, it breaks my site http://i.imgur.com/3pI6k.png but that's not a major problem. PS: Incidentally I just noticed on right on this page: http://i.imgur.com/3pI6k.png didn't check if related. 760828 6 openmail+webkit 2012-11-07 10:15:28 -0800 > PS: Incidentally I just noticed on right on this page … Here's the correct link: http://i.imgur.com/SlFqz.png 763809 7 173496 robert 2012-11-11 03:53:08 -0800 Created attachment 173496 Patch 767112 8 robert 2012-11-14 11:33:16 -0800 Hi Dan, Can you review this please? Thanks.. 775756 9 173496 hyatt 2012-11-26 12:09:53 -0800 Comment on attachment 173496 Patch r=me 777006 10 173496 webkit.review.bot 2012-11-27 11:15:52 -0800 Comment on attachment 173496 Patch Clearing flags on attachment: 173496 Committed r135884: <http://trac.webkit.org/changeset/135884> 777007 11 webkit.review.bot 2012-11-27 11:15:56 -0800 All reviewed patches have been landed. Closing bug. 172750 2012-11-07 03:25:29 -0800 2012-11-07 03:25:29 -0800 Example of the issue. example.html text/html 696 openmail+webkit PCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBYSFRNTCAxLjAgVHJhbnNpdGlvbmFs Ly9FTiIgImh0dHA6Ly93d3cudzMub3JnL1RSL3hodG1sMS9EVEQveGh0bWwxLXRyYW5zaXRpb25h bC5kdGQiPgo8aHRtbCB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMTk5OS94aHRtbCI+CiAgICA8 aGVhZD4KICAgICAgICA8bWV0YSBodHRwLWVxdWl2PSJDb250ZW50LVR5cGUiIGNvbnRlbnQ9InRl eHQvaHRtbDsgY2hhcnNldD1VVEYtOCI+ICAgIAogICAgICAgIDxsaW5rIGhyZWY9J2h0dHA6Ly9m b250cy5nb29nbGVhcGlzLmNvbS9jc3M/ZmFtaWx5PU9wZW4rU2FucycgcmVsPSdzdHlsZXNoZWV0 JyB0eXBlPSd0ZXh0L2Nzcyc+CiAgICAgICAgPHN0eWxlIHR5cGU9InRleHQvY3NzIj4KICAgICAg ICAgICBib2R5IHsKICAgICAgICAgICAgICB3b3JkLXNwYWNpbmc6IDFweDsKICAgICAgICAgICAg ICBmb250LWZhbWlseToiT3BlbiBTYW5zIjsKICAgICAgICAgICAgfQoKICAgICAgICAgICAgLmwt dG9wLW1lbnUgewogICAgICAgICAgICAgIHBvc2l0aW9uOmFic29sdXRlOwogICAgICAgICAgICB9 CiAgICAgICAgPC9zdHlsZT4KICAgIDwvaGVhZD4KICAgIDxib2R5PgogICAgICAgIDxkaXYgY2xh c3M9ImwtdG9wLW1lbnUiPgogICAgICAgICAgYSAmbmJzcDtiCiAgICAgICA8L2Rpdj4KICA8L2Jv ZHk+CjwvaHRtbD4K 173496 2012-11-11 03:53:08 -0800 2012-11-27 11:15:52 -0800 Patch bug-101443-20121111115107.patch text/plain 4351 robert U3VidmVyc2lvbiBSZXZpc2lvbjogMTMyMTMwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZTNiOTI2NDkyYTcwNzQz Y2EzNGVmMzAzMDY1ODZiZDU0NjNlNmI4OS4uODI4ZTkyZjNmMzM2ODgyYjI3YzcyYjRhMTBiZDI4 MGYzMWI3NjEwOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIwIEBACisyMDEyLTExLTExICBSb2Jl cnQgSG9nYW4gIDxyb2JlcnRAd2Via2l0Lm9yZz4KKworICAgICAgICBSRUdSRVNTSU9OIChyMTI1 NTc4KTogd29yZC13cmFwcGluZyBpbiBhYnNvbHV0ZSBwb3NpdGlvbiB3aXRoIG5ic3AsIHdvcmQt c3BhY2luZyBhbmQgY3VzdG9tIGZvbnQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTEwMTQ0MworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q UyEpLgorCisgICAgICAgIExlYWRpbmcgbm9uLWJyZWFraW5nIHNwYWNlIGluIGEgdGV4dCBydW4g c2hvdWxkIGFsd2F5cyBnZXQgd29yZC1zcGFjaW5nIGFwcGxpZWQuIFdoZW4gUmVuZGVyVGV4dAor ICAgICAgICBjYWxjdWxhdGVzIHRoZSBwcmVmZXJyZWQgd2l0aCBvZiBhIHJ1biBvZiB0ZXh0IGl0 IGJyZWFrcyBpdCB1cCBpbnRvIHdvcmQgc2VnbWVudHMgdGhhdCBjb250YWluCisgICAgICAgIG5v IGJyZWFraW5nIHNwYWNlcyBhbmQgY2FsY3VsYXRlcyB0aGUgd2lkdGggb2YgZWFjaC4gU28gd2hl biBjYWxjdWxhdGluZyB0aGUgd2lkdGggb2YgJyZuYnNwO2InIAorICAgICAgICBpdCBuZWVkcyB0 byBhZGQgd29yZC1zcGFjaW5nIHRvIHRoZSBsZWFkaW5nIG5vbi1icmVha2luZyBzcGFjZSBhcyBp dCBkb2VzIG5vdCBnZXQgY29sbGFwc2VkIGF3YXkuCisKKyAgICAgICAgVGVzdDogZmFzdC90ZXh0 L3dvcmQtc3BhY2UtbmJzcC5odG1sCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9XaWR0 aEl0ZXJhdG9yLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OldpZHRoSXRlcmF0b3I6OmFkdmFuY2VJ bnRlcm5hbCk6CisKIDIwMTItMTAtMjIgIE1pa2UgV2VzdCAgPG1rd3N0QGNocm9taXVtLm9yZz4K IAogICAgICAgICAnaW1hZ2UvcGpwZWcnIHNob3VsZCBiZSB0cmVhdGVkIGFzIGFuIGltYWdlIGJ5 IFdlYiBJbnNwZWN0b3IuCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFw aGljcy9XaWR0aEl0ZXJhdG9yLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNz L1dpZHRoSXRlcmF0b3IuY3BwCmluZGV4IDE1Y2E2MTc3Mzk1YTcxMGY1YTIxYjkxNWZmNTc1YjQy YjkxOWExZDAuLjk2NjAwYzczMmZmM2UxN2Q4MGVmMDdkMWUyOGJlNzU5NWY0NzA2YjEgMTAwNjQ0 Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL1dpZHRoSXRlcmF0b3IuY3Bw CisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL1dpZHRoSXRlcmF0b3IuY3Bw CkBAIC0yNDEsNyArMjQxLDcgQEAgaW5saW5lIHVuc2lnbmVkIFdpZHRoSXRlcmF0b3I6OmFkdmFu Y2VJbnRlcm5hbChUZXh0SXRlcmF0b3ImIHRleHRJdGVyYXRvciwgR2x5cGgKIAogICAgICAgICAg ICAgICAgIC8vIEFjY291bnQgZm9yIHdvcmQgc3BhY2luZy4KICAgICAgICAgICAgICAgICAvLyBX ZSBhcHBseSBhZGRpdGlvbmFsIHNwYWNlIGJldHdlZW4gIndvcmRzIiBieSBhZGRpbmcgd2lkdGgg dG8gdGhlIHNwYWNlIGNoYXJhY3Rlci4KLSAgICAgICAgICAgICAgICBpZiAodHJlYXRBc1NwYWNl ICYmIChjaGFyYWN0ZXIgIT0gJ1x0JyB8fCAhbV9ydW4uYWxsb3dUYWJzKCkpICYmIHRleHRJdGVy YXRvci5jdXJyZW50Q2hhcmFjdGVyKCkgJiYgbV9mb250LT53b3JkU3BhY2luZygpKQorICAgICAg ICAgICAgICAgIGlmICh0cmVhdEFzU3BhY2UgJiYgKGNoYXJhY3RlciAhPSAnXHQnIHx8ICFtX3J1 bi5hbGxvd1RhYnMoKSkgJiYgKHRleHRJdGVyYXRvci5jdXJyZW50Q2hhcmFjdGVyKCkgfHwgY2hh cmFjdGVyID09IG5vQnJlYWtTcGFjZSkgJiYgbV9mb250LT53b3JkU3BhY2luZygpKQogICAgICAg ICAgICAgICAgICAgICB3aWR0aCArPSBtX2ZvbnQtPndvcmRTcGFjaW5nKCk7CiAgICAgICAgICAg ICB9IGVsc2UKICAgICAgICAgICAgICAgICBtX2lzQWZ0ZXJFeHBhbnNpb24gPSBmYWxzZTsKZGlm ZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwpp bmRleCA4MDcyNmMxZGYxNDk1ZjBhNjQxYjk3YzMxM2I1OTk4NDMxYWZjNzk2Li4zNGRhYzczYzY3 NzVkOTVmYWNmODFkNDNhNjNlNWExMGU2YWJkMmNmIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9D aGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIw MTItMTEtMTEgIFJvYmVydCBIb2dhbiAgPHJvYmVydEB3ZWJraXQub3JnPgorCisgICAgICAgIFJF R1JFU1NJT04gKHIxMjU1NzgpOiB3b3JkLXdyYXBwaW5nIGluIGFic29sdXRlIHBvc2l0aW9uIHdp dGggbmJzcCwgd29yZC1zcGFjaW5nIGFuZCBjdXN0b20gZm9udAorICAgICAgICBodHRwczovL2J1 Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTAxNDQzCisKKyAgICAgICAgUmV2aWV3ZWQg YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBmYXN0L3RleHQvd29yZC1zcGFjZS1uYnNw LWV4cGVjdGVkLmh0bWw6IEFkZGVkLgorICAgICAgICAqIGZhc3QvdGV4dC93b3JkLXNwYWNlLW5i c3AuaHRtbDogQWRkZWQuCisKIDIwMTItMTAtMjIgIFNoZXJpZmYgQm90ICA8d2Via2l0LnJldmll dy5ib3RAZ21haWwuY29tPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIHJvbGxpbmcgb3V0IHIxMzIx MTkuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0L3RleHQvd29yZC1zcGFjZS1uYnNwLWV4 cGVjdGVkLmh0bWwgYi9MYXlvdXRUZXN0cy9mYXN0L3RleHQvd29yZC1zcGFjZS1uYnNwLWV4cGVj dGVkLmh0bWwKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAw MDAwMDAwMDAwMDAwMDAwMDAwMC4uN2EyOGJhZmJmODlkMjliYTg1NWM0YjFjNTU3NGQwZGFlZTRj YTU4ZQotLS0gL2Rldi9udWxsCisrKyBiL0xheW91dFRlc3RzL2Zhc3QvdGV4dC93b3JkLXNwYWNl LW5ic3AtZXhwZWN0ZWQuaHRtbApAQCAtMCwwICsxLDE5IEBACis8IURPQ1RZUEUgaHRtbD4KKzxo dG1sPgorICAgIDxoZWFkPgorICAgICAgICA8c3R5bGUgdHlwZT0idGV4dC9jc3MiPgorICAgICAg ICAgICAgZGl2IHsKKyAgICAgICAgICAgICAgd29yZC1zcGFjaW5nOiAyMHB4OworICAgICAgICAg ICAgICBmbG9hdDogbGVmdDsKKyAgICAgICAgICAgICAgYm9yZGVyOiBzb2xpZDsKKyAgICAgICAg ICAgICAgY2xlYXI6IGJvdGg7CisgICAgICAgICAgICB9CisgICAgICAgIDwvc3R5bGU+CisgICAg PC9oZWFkPgorICAgIDxib2R5PgorICAgICAgICA8cD5UaGUgbGluZXMgYmVsb3cgc2hvdWxkIGJl IHRoZSBzYW1lIHdpZHRoLjwvcD4KKyAgICAgICAgPGRpdj5hJm5ic3A7Jm5ic3A7YjwvZGl2Pgor ICAgICAgICA8ZGl2PmEmbmJzcDsmbmJzcDtiPC9kaXY+CisgIDwvYm9keT4KKzwvaHRtbD4KKwpk aWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC90ZXh0L3dvcmQtc3BhY2UtbmJzcC5odG1sIGIv TGF5b3V0VGVzdHMvZmFzdC90ZXh0L3dvcmQtc3BhY2UtbmJzcC5odG1sCm5ldyBmaWxlIG1vZGUg MTAwNjQ0CmluZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLmY0 NzQ4NjlkMDNmNTJhZjU4NjdjNmM3NzA0MGQzZGU5ZWZmOTVlYTIKLS0tIC9kZXYvbnVsbAorKysg Yi9MYXlvdXRUZXN0cy9mYXN0L3RleHQvd29yZC1zcGFjZS1uYnNwLmh0bWwKQEAgLTAsMCArMSwx OSBAQAorPCFET0NUWVBFIGh0bWw+Cis8aHRtbD4KKyAgICA8aGVhZD4KKyAgICAgICAgPHN0eWxl IHR5cGU9InRleHQvY3NzIj4KKyAgICAgICAgICAgIGRpdiB7CisgICAgICAgICAgICAgIHdvcmQt c3BhY2luZzogMjBweDsKKyAgICAgICAgICAgICAgZmxvYXQ6IGxlZnQ7CisgICAgICAgICAgICAg IGJvcmRlcjogc29saWQ7CisgICAgICAgICAgICAgIGNsZWFyOiBib3RoOworICAgICAgICAgICAg fQorICAgICAgICA8L3N0eWxlPgorICAgIDwvaGVhZD4KKyAgICA8Ym9keT4KKyAgICAgICAgPHA+ VGhlIGxpbmVzIGJlbG93IHNob3VsZCBiZSB0aGUgc2FtZSB3aWR0aC48L3A+CisgICAgICAgIDxk aXY+YSAmbmJzcDtiPC9kaXY+CisgICAgICAgIDxkaXY+YSZuYnNwOyZuYnNwO2I8L2Rpdj4KKyAg PC9ib2R5PgorPC9odG1sPgorCg==