47374 2010-10-07 13:09:22 -0700 REGRESSION (r47440): Printing Mail messages with large fonts does not fill page 2010-10-07 14:02:25 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All All RESOLVED FIXED InRadar, Regression P1 Normal --- 1 mitz webkit-unassigned oldest_to_newest 291214 0 mitz 2010-10-07 13:09:22 -0700 <rdar://problem/8142645> In the legacy (paint-time pagination) printing model, when there are overlapping lines, adjustPageHeightDeprecated() is not idempotent, since the truncation point imposed by line n, namely the top of line n, is above the bottom of line n-1, so in the next round, line n-1 moves the truncation point up again. Patch forthcoming. 291217 1 70141 mitz 2010-10-07 13:14:49 -0700 Created attachment 70141 Ignore the part of the line that overlaps the next line 291238 2 mitz 2010-10-07 14:02:25 -0700 Fixed in <http://trac.webkit.org/projects/webkit/changeset/69336>. 70141 2010-10-07 13:14:49 -0700 2010-10-07 13:25:01 -0700 Ignore the part of the line that overlaps the next line 8142645_r1.diff text/plain 2125 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA2OTMzMykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTkgQEAKKzIwMTAtMTAtMDcgIERhbiBCZXJuc3RlaW4gIDxtaXR6QGFwcGxlLmNv bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICA8cmRh cjovL3Byb2JsZW0vODE0MjY0NT4gUkVHUkVTU0lPTiAocjQ3NDQwKTogUHJpbnRpbmcgTWFpbCBt ZXNzYWdlcyB3aXRoIGxhcmdlIGZvbnRzIGRvZXMgbm90IGZpbGwgcGFnZQorICAgICAgICBodHRw czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDczNzQKKworICAgICAgICBJbiB0 aGUgbGVnYWN5IChwYWludC10aW1lIHBhZ2luYXRpb24pIHByaW50aW5nIG1vZGVsLCB3aGVuIHRo ZXJlIGFyZSBvdmVybGFwcGluZyBsaW5lcywKKyAgICAgICAgYWRqdXN0UGFnZUhlaWdodERlcHJl Y2F0ZWQoKSB3YXMgbm90IGlkZW1wb3RlbnQsIHNpbmNlIHRoZSB0cnVuY2F0aW9uIHBvaW50IGlt cG9zZWQgYnkgbGluZSBuLAorICAgICAgICBuYW1lbHkgdGhlIHRvcCBvZiBsaW5lIG4sIHdhcyBh Ym92ZSB0aGUgYm90dG9tIG9mIGxpbmUgbi0xLCBzbyBpbiB0aGUgbmV4dCByb3VuZCwgbGluZSBu LTEKKyAgICAgICAgbW92ZWQgdGhlIHRydW5jYXRpb24gcG9pbnQgdXAgYWdhaW4uCisKKyAgICAg ICAgKiByZW5kZXJpbmcvUmVuZGVyTGluZUJveExpc3QuY3BwOgorICAgICAgICAoV2ViQ29yZTo6 UmVuZGVyTGluZUJveExpc3Q6OnBhaW50KTogQ29uc2lkZXIgdGhlIHRvcCBvZiB0aGUgbmV4dCBs aW5lIHdoZW4gZGVjaWRpbmcgaWYgdGhlCisgICAgICAgIGN1cnJlbnQgbGluZSBleHRlbmRzIGJl bG93IHRoZSB0cnVuY2F0aW9uIHBvaW50LgorCiAyMDEwLTEwLTA3ICBLd2FuZyBZdWwgU2VvICA8 c2t5dWxAY29tcGFueTEwMC5uZXQ+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgS2VudCBUYW11cmEu CkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMaW5lQm94TGlzdC5jcHAKPT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PQotLS0gV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTGluZUJveExpc3QuY3BwCShyZXZpc2lvbiA2 OTMyNykKKysrIFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxpbmVCb3hMaXN0LmNwcAkod29ya2lu ZyBjb3B5KQpAQCAtMjAzLDYgKzIwMywxMCBAQCB2b2lkIFJlbmRlckxpbmVCb3hMaXN0OjpwYWlu dChSZW5kZXJCb3hNCiAgICAgICAgICAgICB9CiAgICAgICAgICAgICBpZiAoYm90dG9tRm9yUGFn aW5hdGlvbkNoZWNrIC0gdG9wRm9yUGFnaW5hdGlvbkNoZWNrIDw9IHYtPnByaW50UmVjdCgpLmhl aWdodCgpKSB7CiAgICAgICAgICAgICAgICAgaWYgKHR5ICsgYm90dG9tRm9yUGFnaW5hdGlvbkNo ZWNrID4gdi0+cHJpbnRSZWN0KCkuYm90dG9tKCkpIHsKKyAgICAgICAgICAgICAgICAgICAgaWYg KFJvb3RJbmxpbmVCb3gqIG5leHRSb290Qm94ID0gY3Vyci0+cm9vdCgpLT5uZXh0Um9vdEJveCgp KQorICAgICAgICAgICAgICAgICAgICAgICAgYm90dG9tRm9yUGFnaW5hdGlvbkNoZWNrID0gbWlu KGJvdHRvbUZvclBhZ2luYXRpb25DaGVjaywgbWluKG5leHRSb290Qm94LT50b3BWaXNpYmxlT3Zl cmZsb3coKSwgbmV4dFJvb3RCb3gtPmxpbmVUb3AoKSkpOworICAgICAgICAgICAgICAgIH0KKyAg ICAgICAgICAgICAgICBpZiAodHkgKyBib3R0b21Gb3JQYWdpbmF0aW9uQ2hlY2sgPiB2LT5wcmlu dFJlY3QoKS5ib3R0b20oKSkgewogICAgICAgICAgICAgICAgICAgICBpZiAodHkgKyB0b3BGb3JQ YWdpbmF0aW9uQ2hlY2sgPCB2LT50cnVuY2F0ZWRBdCgpKQogICAgICAgICAgICAgICAgICAgICAg ICAgdi0+c2V0QmVzdFRydW5jYXRlZEF0KHR5ICsgdG9wRm9yUGFnaW5hdGlvbkNoZWNrLCByZW5k ZXJlcik7CiAgICAgICAgICAgICAgICAgICAgIC8vIElmIHdlIHdlcmUgYWJsZSB0byB0cnVuY2F0 ZSwgZG9uJ3QgcGFpbnQuCg==