[Webkit-unassigned] [Bug 10729] Very short plain text documents have a wider top margin

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=10729


ap at nypop.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |darin at apple.com




------- Comment #1 from ap at nypop.com  2006-09-04 11:46 PDT -------
(In reply to comment #0)
> The decoder changes in r16175 caused fast/text/plain-text-line-breaks.html to
> fail. The new result seems to have the correct layout.

  Oops, I could have sworn that I checked for changes in test results.

> I don't understand the role of the recalcStyle() call in write() (in
> particular, whether it was intended only for the visually ordered case), so I
> don't know if adding it in the corresponding places in endIfNotLoading() and in
> write(const DeprecatedString& str) is the correct fix.

  Neither do I, but there is no reason for StreamingTextDecoder to buffer these
bytes anyway - there is no chance that a BOM will appear from nowhere on a
second pass. Darin's upcoming decoding patch might even already fix this,
actually.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.