Fix for 3790936, make the unicode-breaking on the layout tests match Panther.
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Oct 2004 20:58:12 +0000 (20:58 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Oct 2004 20:58:12 +0000 (20:58 +0000)
        Reviewed by kocienda

        * khtml/rendering/break_lines.cpp:
        (khtml::isBreakable):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7867 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/rendering/break_lines.cpp

index 91ae6699b11bb480df762f615c7c3ccbad2c63cb..afee5f59f6aad5b76a64bc051cfb6d26cb8da781 100644 (file)
@@ -1,3 +1,12 @@
+2004-10-20  David Hyatt  <hyatt@apple.com>
+
+       Fix for 3790936, make the unicode-breaking on the layout tests match Panther.
+       
+        Reviewed by kocienda
+
+        * khtml/rendering/break_lines.cpp:
+        (khtml::isBreakable):
+
 2004-10-20  Darin Adler  <darin@apple.com>
 
         Reviewed by John.
index a1f06869e2994ce3315694716447f05c36bdd39d..ef8df0f01c029be1a9deaf1f2dd7f064952a93c3 100644 (file)
@@ -84,8 +84,12 @@ bool isBreakable(const QChar *s, int pos, int len, bool breakNBSP)
     const QChar *c = s+pos;
     unsigned short ch = c->unicode();
 
-    // Newlines are always breakable, as are non-breaking spaces when breakNBSP is true.
-    if (ch == '\n' || (breakNBSP && ch == 0xa0))
+    // Newlines and spaces are always breakable, as are non-breaking spaces when breakNBSP is true.
+    // FIXME: Tiger is not allowing breaks on spaces after bullet characters like &#183;.  We don't know
+    // at the moment whether this behavior is correct or not.  Since Tiger is also not allowing breaks on spaces
+    // after hyphen-like characters, this does not seem ideal for the Web.  Therefore for now we override space
+    // characters up front and bypass the Unicode line breaking routines.
+    if (ch == '\n' || ch == ' ' || (breakNBSP && ch == 0xa0))
         return true;
 
     // If current character, or the previous character aren't simple latin1 then