[webkit-reviews] review requested: [Bug 3838] Text runs unnecessarily split at EN L boundaries : [Attachment 2830] Proposed patch

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Wed Jul 6 14:28:57 PDT 2005


opendarwin.org at mitzpettel.com has asked  for review:
Bug 3838: Text runs unnecessarily split at EN L boundaries
http://bugzilla.opendarwin.org/show_bug.cgi?id=3838

Attachment 2830: Proposed patch
http://bugzilla.opendarwin.org/attachment.cgi?id=2830&action=edit

------- Additional Comments from opendarwin.org at mitzpettel.com
The patch changes the layout trees of 136 layout tests, by joining runs.

I have rendered each of the 136 into 800x800 PNG files, with and without the
patch, and then compared the results. Differences were found in only 2 out of
the 136 cases:
  css1/box_properties/clear_float.html
and
  css1/box_properties/float_on_text_elements.html.
The differences are minimal and are due to the ability to better distribute the
spacing when justifying a longer run. See the attachments for one comparison.
The other one is similar.



More information about the webkit-reviews mailing list