Commit 98726dac authored by weinig's avatar weinig
Browse files

LayoutTests:

        Reviewed by Hyatt.

        - test for http://bugs.webkit.org/show_bug.cgi?id=13072
          REGRESSION (r15617): white-space: pre-wrap breaks off the last character of a wide word

        * fast/text/whitespace/pre-wrap-last-char-expected.checksum: Added.
        * fast/text/whitespace/pre-wrap-last-char-expected.png: Added.
        * fast/text/whitespace/pre-wrap-last-char-expected.txt: Added.
        * fast/text/whitespace/pre-wrap-last-char.html: Added.

WebCore:

        Reviewed by Hyatt.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13072
          REGRESSION (r15617): white-space: pre-wrap breaks off the last character of a wide word

        Test: fast/text/whitespace/pre-wrap-last-char.html

        * rendering/bidi.cpp:
        (WebCore::RenderBlock::findNextLineBreak): Undid the change from r15617.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@20200 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent f56c4413
2007-03-14 Mitz Pettel <mitz@webkit.org>
Reviewed by Hyatt.
- test for http://bugs.webkit.org/show_bug.cgi?id=13072
REGRESSION (r15617): white-space: pre-wrap breaks off the last character of a wide word
* fast/text/whitespace/pre-wrap-last-char-expected.checksum: Added.
* fast/text/whitespace/pre-wrap-last-char-expected.png: Added.
* fast/text/whitespace/pre-wrap-last-char-expected.txt: Added.
* fast/text/whitespace/pre-wrap-last-char.html: Added.
2007-03-14 Justin Garcia <justin.garcia@apple.com>
 
Reviewed by darin
5b9d528aff8834f11eb0b7f087e43989
\ No newline at end of file
layer at (0,0) size 800x600
RenderView at (0,0) size 800x600
layer at (0,0) size 800x600
RenderBlock {HTML} at (0,0) size 800x600
RenderBody {BODY} at (8,8) size 784x584
RenderBlock {P} at (0,0) size 784x36
RenderText {#text} at (0,0) size 53x18
text run at (0,0) width 53: "Test for "
RenderInline {I} at (0,0) size 781x36
RenderInline {A} at (0,0) size 301x18 [color=#0000EE]
RenderText {#text} at (53,0) size 301x18
text run at (53,0) width 301: "http://bugs.webkit.org/show_bug.cgi?id=13072"
RenderText {#text} at (354,0) size 781x36
text run at (354,0) width 4: " "
text run at (358,0) width 423: "REGRESSION (r15617): white-space: pre-wrap breaks off the last"
text run at (0,18) width 160: "character of a wide word"
RenderText {#text} at (160,18) size 4x18
text run at (160,18) width 4: "."
RenderBlock {P} at (0,52) size 784x18
RenderText {#text} at (0,0) size 227x18
text run at (0,0) width 227: "The following line should not wrap."
RenderBlock {DIV} at (0,86) size 150x18 [bgcolor=#D3D3D3]
RenderText {#text} at (0,0) size 150x18
text run at (0,0) width 150: "abcdefghijklmnopqrstuvwxyz"
<html>
<head>
<title></title>
</head>
<body>
<p>
Test for <i><a href="http://bugs.webkit.org/show_bug.cgi?id=13072">http://bugs.webkit.org/show_bug.cgi?id=13072</a>
REGRESSION (r15617): white-space: pre-wrap breaks off the last character of a wide word</i>.
</p>
<p>
The following line should not wrap.
</p>
<div style="white-space: pre-wrap; width: 150px; background-color: lightgray;">abcdefghijklmnopqrstuvwxyz</div>
</body>
2007-03-14 Mitz Pettel <mitz@webkit.org>
Reviewed by Hyatt.
- fix http://bugs.webkit.org/show_bug.cgi?id=13072
REGRESSION (r15617): white-space: pre-wrap breaks off the last character of a wide word
Test: fast/text/whitespace/pre-wrap-last-char.html
* rendering/bidi.cpp:
(WebCore::RenderBlock::findNextLineBreak): Undid the change from r15617.
2007-03-14 David Hyatt <hyatt@apple.com>
 
Add asserts to help catch double refs and double derefs of CachedResources.
......
......@@ -2582,7 +2582,7 @@ BidiIterator RenderBlock::findNextLineBreak(BidiIterator &start, BidiState &bidi
if (lBreak == start && !lBreak.obj->isBR()) {
// we just add as much as possible
if (shouldPreserveNewline(this)) {
if (style()->whiteSpace() == PRE) {
// FIXME: Don't really understand this case.
if (pos != 0) {
lBreak.obj = o;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment