Title: [128965] trunk/Source/WebCore
Revision
128965
Author
[email protected]
Date
2012-09-18 22:26:16 -0700 (Tue, 18 Sep 2012)

Log 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.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (128964 => 128965)


--- trunk/Source/WebCore/ChangeLog	2012-09-19 04:21:50 UTC (rev 128964)
+++ trunk/Source/WebCore/ChangeLog	2012-09-19 05:26:16 UTC (rev 128965)
@@ -1,3 +1,15 @@
+2012-09-18  Kenichi Ishibashi  <[email protected]>
+
+        [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  <[email protected]>
 
         Check settings before registering AVFoundation media engine.

Modified: trunk/Source/WebCore/platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp (128964 => 128965)


--- trunk/Source/WebCore/platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp	2012-09-19 04:21:50 UTC (rev 128964)
+++ trunk/Source/WebCore/platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp	2012-09-19 05:26:16 UTC (rev 128965)
@@ -155,7 +155,8 @@
         UChar32 character;
         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))
             character = zeroWidthSpace;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to