5593 2005-11-02 01:00:09 -0800 CSS2: pre-wrap needs to break only after all white-space 2005-11-03 13:05:07 -0800 1 1 1 Unclassified WebKit Layout and Rendering 420+ Mac OS X 10.4 RESOLVED FIXED P2 Normal --- 1 hyatt hyatt oldest_to_newest 23512 0 hyatt 2005-11-02 01:00:09 -0800 In the attached testcase, you will see white-space at the start of lines. This basically shouldn't happen unless that space is after a line that broke cleanly (or the first line). Breaking is occurring in the middle of runs of white-space, and it should only be allowed to occur at the end of the run of white-space. 23513 1 4561 hyatt 2005-11-02 01:00:50 -0800 Created attachment 4561 Test Case 23633 2 4585 hyatt 2005-11-03 12:03:18 -0800 Created attachment 4585 Patch that collapses white-space for pre-wrap at the start of lines that didn't break cleanly. 23634 3 4585 adele 2005-11-03 12:14:51 -0800 Comment on attachment 4585 Patch that collapses white-space for pre-wrap at the start of lines that didn't break cleanly. r=me 4561 2005-11-02 01:00:50 -0800 2005-11-02 01:00:50 -0800 Test Case pre-wrap.html text/html 236 hyatt PGh0bWw+CjxoZWFkPgo8dGl0bGU+UHJlLVdyYXAgVGVzdCBQYWdlPC90aXRsZT4KPC9oZWFkPgo8 Ym9keT4KPGgxPlRlc3Qgb2YgdGhlIHdyYXAgYXR0cmlidXRlIG9uIHByZTwvaDE+CjxwcmUgY29u dGVudGVkaXRhYmxlIHdyYXAgc3R5bGU9ImJvcmRlcjoycHggc29saWQgcmVkOyB3aWR0aDoxMDBw eCI+ClRoaXMgICB0ZXh0ICAgICB3aWxsIHdyYXAKYW5kICAgZml0ICAgd2l0aGluIHRoZQpib3gu CjwvcHJlPgo= 4585 2005-11-03 12:03:18 -0800 2005-11-03 12:14:51 -0800 Patch that collapses white-space for pre-wrap at the start of lines that didn't break cleanly. patch.txt text/plain 3288 hyatt SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvY3ZzL3Jvb3QvV2ViQ29yZS9D aGFuZ2VMb2csdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEuMzMxCmRpZmYgLXUgLXAgLXIxLjMzMSBD aGFuZ2VMb2cKLS0tIENoYW5nZUxvZwkzIE5vdiAyMDA1IDIwOjAwOjUxIC0wMDAwCTEuMzMxCisr KyBDaGFuZ2VMb2cJMyBOb3YgMjAwNSAyMDowMzo0OCAtMDAwMApAQCAtMSwzICsxLDE1IEBACisy MDA1LTExLTAzICBEYXZpZCBIeWF0dCAgPGh5YXR0QGFwcGxlLmNvbT4KKworICAgICAgICBSZXZp ZXdlZCBieSBhZGVsZQorCisJZmFzdC90ZXh0L3doaXRlc3BhY2UvcHJlLXdyYXAuaHRtbAorCisg ICAgICAgICoga2h0bWwvcmVuZGVyaW5nL2JpZGkuY3BwOgorICAgICAgICAoa2h0bWw6OnNraXBO b25CcmVha2luZ1NwYWNlKToKKyAgICAgICAgKGtodG1sOjpzaG91bGRDb2xsYXBzZVdoaXRlU3Bh Y2UpOgorICAgICAgICAoa2h0bWw6OlJlbmRlckJsb2NrOjpza2lwV2hpdGVzcGFjZSk6CisgICAg ICAgICoga2h0bWwvcmVuZGVyaW5nL3JlbmRlcl9ibG9jay5oOgorCiAyMDA1LTExLTAzICBBZGVs ZSBQZXRlcnNvbiAgPGFkZWxlQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXJp bi4KSW5kZXg6IGtodG1sL3JlbmRlcmluZy9iaWRpLmNwcAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAv Y3ZzL3Jvb3QvV2ViQ29yZS9raHRtbC9yZW5kZXJpbmcvYmlkaS5jcHAsdgpyZXRyaWV2aW5nIHJl dmlzaW9uIDEuMTU0CmRpZmYgLXUgLXAgLXIxLjE1NCBraHRtbC9yZW5kZXJpbmcvYmlkaS5jcHAK LS0tIGtodG1sL3JlbmRlcmluZy9iaWRpLmNwcAkyIE5vdiAyMDA1IDA4OjUyOjQ0IC0wMDAwCTEu MTU0CisrKyBraHRtbC9yZW5kZXJpbmcvYmlkaS5jcHAJMyBOb3YgMjAwNSAyMDowMzo0OSAtMDAw MApAQCAtMTg3Miw3ICsxODcyLDcgQEAgYm9vbCBSZW5kZXJCbG9jazo6bWF0Y2hlZEVuZExpbmUo Y29uc3QgQgogCiBzdGF0aWMgY29uc3QgdXNob3J0IG5vbkJyZWFraW5nU3BhY2UgPSAweGEwOwog Ci1pbmxpbmUgYm9vbCBSZW5kZXJCbG9jazo6c2tpcE5vbkJyZWFraW5nU3BhY2UoQmlkaUl0ZXJh dG9yICZpdCkKK3N0YXRpYyBpbmxpbmUgYm9vbCBza2lwTm9uQnJlYWtpbmdTcGFjZShCaWRpSXRl cmF0b3IgJml0KQogewogICAgIGlmIChpdC5vYmotPnN0eWxlKCktPm5ic3BNb2RlKCkgIT0gU1BB Q0UgfHwgaXQuY3VycmVudCgpLnVuaWNvZGUoKSAhPSBub25CcmVha2luZ1NwYWNlKQogICAgICAg ICByZXR1cm4gZmFsc2U7CkBAIC0xODg4LDYgKzE4ODgsMTEgQEAgaW5saW5lIGJvb2wgUmVuZGVy QmxvY2s6OnNraXBOb25CcmVha2luZwogICAgIHJldHVybiB0cnVlOwogfQogCitzdGF0aWMgaW5s aW5lIGJvb2wgc2hvdWxkQ29sbGFwc2VXaGl0ZVNwYWNlKGNvbnN0IFJlbmRlclN0eWxlKiBzdHls ZSkKK3sKKyAgICByZXR1cm4gc3R5bGUtPmNvbGxhcHNlV2hpdGVTcGFjZSgpIHx8IChzdHlsZS0+ d2hpdGVTcGFjZSgpID09IFBSRV9XUkFQICYmICghaXNMaW5lRW1wdHkgfHwgIXByZXZpb3VzTGlu ZUJyb2tlQ2xlYW5seSkpOworfQorCiBpbnQgUmVuZGVyQmxvY2s6OnNraXBXaGl0ZXNwYWNlKEJp ZGlJdGVyYXRvciAmaXQsIEJpZGlTdGF0ZSAmYmlkaSkKIHsKICAgICAvLyBGSVhNRTogVGhlIGVu dGlyZSBjb25jZXB0IG9mIHRoZSBza2lwV2hpdGVzcGFjZSBmdW5jdGlvbiBpcyBmbGF3ZWQsIHNp bmNlIHdlIHJlYWxseSBuZWVkIHRvIGJlIGJ1aWxkaW5nCkBAIC0xODk1LDcgKzE5MDAsNyBAQCBp bnQgUmVuZGVyQmxvY2s6OnNraXBXaGl0ZXNwYWNlKEJpZGlJdGVyCiAgICAgLy8gZWxlbWVudHMg cXVpdGUgcmlnaHQuICBJbiBvdGhlciB3b3Jkcywgd2UgbmVlZCB0byBidWlsZCB0aGlzIGZ1bmN0 aW9uJ3Mgd29yayBpbnRvIHRoZSBub3JtYWwgbGluZQogICAgIC8vIG9iamVjdCBpdGVyYXRpb24g cHJvY2Vzcy4KICAgICBpbnQgdyA9IGxpbmVXaWR0aChtX2hlaWdodCk7Ci0gICAgd2hpbGUgKCFp dC5hdEVuZCgpICYmIChpdC5vYmotPmlzSW5saW5lRmxvdygpIHx8IChpdC5vYmotPnN0eWxlKCkt PmNvbGxhcHNlV2hpdGVTcGFjZSgpICYmICFpdC5vYmotPmlzQlIoKSAmJgorICAgIHdoaWxlICgh aXQuYXRFbmQoKSAmJiAoaXQub2JqLT5pc0lubGluZUZsb3coKSB8fCAoc2hvdWxkQ29sbGFwc2VX aGl0ZVNwYWNlKGl0Lm9iai0+c3R5bGUoKSkgJiYgIWl0Lm9iai0+aXNCUigpICYmCiAgICAgICAg ICAgKGl0LmN1cnJlbnQoKSA9PSAnICcgfHwgaXQuY3VycmVudCgpID09ICdcdCcgfHwgKCFpdC5v YmotPnN0eWxlKCktPnByZXNlcnZlTmV3bGluZSgpICYmIGl0LmN1cnJlbnQoKSA9PSAnXG4nKSB8 fAogICAgICAgICAgIGl0LmN1cnJlbnQoKS51bmljb2RlKCkgPT0gU09GVF9IWVBIRU4gfHwgc2tp cE5vbkJyZWFraW5nU3BhY2UoaXQpIHx8IGl0Lm9iai0+aXNGbG9hdGluZ09yUG9zaXRpb25lZCgp KSkpKSB7CiAgICAgICAgIGlmIChpdC5vYmotPmlzRmxvYXRpbmdPclBvc2l0aW9uZWQoKSkgewpJ bmRleDoga2h0bWwvcmVuZGVyaW5nL3JlbmRlcl9ibG9jay5oCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZpbGU6 IC9jdnMvcm9vdC9XZWJDb3JlL2todG1sL3JlbmRlcmluZy9yZW5kZXJfYmxvY2suaCx2CnJldHJp ZXZpbmcgcmV2aXNpb24gMS43MQpkaWZmIC11IC1wIC1yMS43MSBraHRtbC9yZW5kZXJpbmcvcmVu ZGVyX2Jsb2NrLmgKLS0tIGtodG1sL3JlbmRlcmluZy9yZW5kZXJfYmxvY2suaAkyIE5vdiAyMDA1 IDA4OjUyOjQ1IC0wMDAwCTEuNzEKKysrIGtodG1sL3JlbmRlcmluZy9yZW5kZXJfYmxvY2suaAkz IE5vdiAyMDA1IDIwOjAzOjQ5IC0wMDAwCkBAIC0yNzAsNyArMjcwLDYgQEAgcHJvdGVjdGVkOgog cHJpdmF0ZToKICAgICBET006OlBvc2l0aW9uIHBvc2l0aW9uRm9yQm94KElubGluZUJveCAqYm94 LCBib29sIHN0YXJ0PXRydWUpIGNvbnN0OwogICAgIERPTTo6UG9zaXRpb24gcG9zaXRpb25Gb3JS ZW5kZXJlcihSZW5kZXJPYmplY3QgKnJlbmRlcmVyLCBib29sIHN0YXJ0PXRydWUpIGNvbnN0Owot ICAgIGJvb2wgc2tpcE5vbkJyZWFraW5nU3BhY2UoQmlkaUl0ZXJhdG9yICZpdCk7CiAgICAgCiBw cm90ZWN0ZWQ6CiAgICAgc3RydWN0IEZsb2F0aW5nT2JqZWN0IHsK