Changeset 128965 in webkit

Timestamp:

Sep 18, 2012 10:26:16 PM (12 years ago)

Author:

bashi@chromium.org

Message:

[Chromium] Don't treat tab as spaces for word-end in HarfBuzzShaper
​https://bugs.webkit.org/show_bug.cgi?id=97068

Reviewed by Yuta Kitamura.

No new tests. fast/text/wide-zero-width-space.html should cover this change.

• platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp:

(WebCore::normalizeCharacters): Don't treat tab as space.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-09-18  Kenichi Ishibashi  <bashi@chromium.org>
+
+        [Chromium] Don't treat tab as spaces for word-end in HarfBuzzShaper
+        https://bugs.webkit.org/show_bug.cgi?id=97068
+
+        Reviewed by Yuta Kitamura.
+
+        No new tests. fast/text/wide-zero-width-space.html should cover this change.
+
+        * platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp:
+        (WebCore::normalizeCharacters): Don't treat tab as space.
+
 2012-09-18  Eric Carlson  <eric.carlson@apple.com>
 

trunk/Source/WebCore/platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp

@@ -156 +156 @@
         int nextPosition = position;
         U16_NEXT(source, nextPosition, length, character);
-        if (Font::treatAsSpace(character))
+        // Don't normalize tabs as they are not treated as spaces for word-end
+        if (Font::treatAsSpace(character) && character != '\t')
             character = ' ';
         else if (Font::treatAsZeroWidthSpaceInComplexScript(character))