Changeset 29915 in webkit

Timestamp:

Feb 1, 2008, 10:06:40 AM (17 years ago)

Author:

mitz@apple.com

Message:

Reviewed by Darin Adler.

• switch line breaking from using Carbon Unicode Utilities to using ICU on Leopard <​http://bugs.webkit.org/show_bug.cgi?id=4628>

fast/text/international/thai-line-breaks.html results do not change.

• rendering/break_lines.cpp: (WebCore::nextBreakablePosition): Changed to use Carbon only on Tiger.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/break_lines.cpp (modified) (3 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2008-02-01  Dan Bernstein  <mitz@apple.com>
+
+        Reviewed by Darin Adler.
+
+        - switch line breaking from using Carbon Unicode Utilities to using ICU
+          on Leopard
+          <http://bugs.webkit.org/show_bug.cgi?id=4628>
+
+        fast/text/international/thai-line-breaks.html results do not change.
+
+        * rendering/break_lines.cpp:
+        (WebCore::nextBreakablePosition): Changed to use Carbon only on Tiger.
+
 2008-02-01  Rodney Dawes  <dobey@wayofthemonkey.com>
 

trunk/WebCore/rendering/break_lines.cpp

@@ -63 +63 @@
 }
 
-#if PLATFORM(MAC)
+#ifdef BUILDING_ON_TIGER
 static inline TextBreakLocatorRef lineBreakLocator()
 {
@@ -74 +74 @@
 int nextBreakablePosition(const UChar* str, int pos, int len, bool treatNoBreakSpaceAsBreak)
 {
-#if !PLATFORM(MAC)
+#ifndef BUILDING_ON_TIGER
     TextBreakIterator* breakIterator = 0;
 #endif
@@ -88 +88 @@
         if (needsLineBreakIterator(ch) || needsLineBreakIterator(lastCh)) {
             if (nextBreak < i && i) {
-#if !PLATFORM(MAC)
+#ifndef BUILDING_ON_TIGER
                 if (!breakIterator)
                     breakIterator = lineBreakIterator(str, len);