Add performance test for asking the platform for a font for U+2060 WORD JOINER
authormmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Apr 2017 17:57:26 +0000 (17:57 +0000)
committermmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Apr 2017 17:57:26 +0000 (17:57 +0000)
https://bugs.webkit.org/show_bug.cgi?id=170842

Reviewed by Tim Horton.

PerformanceTests:

* Layout/word-joiner.html: Added.

Source/WebCore:

No new tests because there is no behavior change.

* platform/graphics/WidthCache.h:
(WebCore::WidthCache::add):
(WebCore::WidthCache::addSlowCase):

Source/WTF:

* wtf/unicode/CharacterNames.h:

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

PerformanceTests/ChangeLog
PerformanceTests/Layout/word-joiner.html [new file with mode: 0644]
Source/WTF/ChangeLog
Source/WTF/wtf/unicode/CharacterNames.h
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/WidthCache.h

index f884aa8..8860b04 100644 (file)
@@ -1,3 +1,12 @@
+2017-04-14  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Add performance test for asking the platform for a font for U+2060 WORD JOINER
+        https://bugs.webkit.org/show_bug.cgi?id=170842
+
+        Reviewed by Tim Horton.
+
+        * Layout/word-joiner.html: Added.
+
 2017-04-03  Saam Barati  <sbarati@apple.com>
 
         Add a new test to ARES6
diff --git a/PerformanceTests/Layout/word-joiner.html b/PerformanceTests/Layout/word-joiner.html
new file mode 100644 (file)
index 0000000..eedceb5
--- /dev/null
@@ -0,0 +1,44 @@
+<!DOCTYPE html>
+<html>
+<head>
+<script src="../resources/runner.js"></script>
+</head>
+</head>
+<body>
+    <pre id="log"></pre>
+    <div id="target" style="width: 300px; display: block; font-family: 'Zapfino';">
+    </div>
+    <script>
+        var target = document.getElementById("target");
+        var style = target.style;
+        var locales = ["ar", "hy", "as", "iu", "ab", "hi", "am", "ka", "el", "gu", "pa", "ko", "ko", "he", "kn", "ja", "km", "lo", "aa", "ml", "my", "or", "si", "ta", "te", "dv", "th", "bo", "zh_hk", "ii"];
+
+        for (var i = 0; i < locales.length; ++i) {
+            var span = document.createElement("span");
+            span.className = "tester";
+            var joiner = document.createTextNode(String.fromCharCode(0x2060));
+            span.appendChild(joiner);
+            target.appendChild(span);
+        }
+
+        function updateLanguages() {
+            if (window.internals)
+                internals.invalidateFontCache();
+            var testers = document.getElementsByClassName("tester");
+            for (var i = 0; i < testers.length; ++i) {
+                testers[i].lang = locales[i];
+            }
+        }
+        
+        updateLanguages();
+
+        function test() {
+            updateLanguages();
+            var testers = document.getElementsByClassName("tester");
+            testers[testers.length - 1].getClientRects();
+        }
+
+        PerfTestRunner.measureRunsPerSecond({ run: test });
+    </script>
+</body>
+</html>
\ No newline at end of file
index f148109..04fa9a9 100644 (file)
@@ -1,3 +1,12 @@
+2017-04-14  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Add performance test for asking the platform for a font for U+2060 WORD JOINER
+        https://bugs.webkit.org/show_bug.cgi?id=170842
+
+        Reviewed by Tim Horton.
+
+        * wtf/unicode/CharacterNames.h:
+
 2017-04-13  Yusuke Suzuki  <utatane.tea@gmail.com>
 
         [JSC] Date.parse should accept wider range of representation
index 1c415be..1bdd060 100644 (file)
@@ -92,6 +92,7 @@ const UChar whiteBullet = 0x25E6;
 const UChar whiteCircle = 0x25CB;
 const UChar whiteSesameDot = 0xFE46;
 const UChar whiteUpPointingTriangle = 0x25B3;
+const UChar wordJoiner = 0x2060;
 const UChar yenSign = 0x00A5;
 const UChar zeroWidthJoiner = 0x200D;
 const UChar zeroWidthNoBreakSpace = 0xFEFF;
@@ -154,6 +155,7 @@ using WTF::Unicode::whiteBullet;
 using WTF::Unicode::whiteCircle;
 using WTF::Unicode::whiteSesameDot;
 using WTF::Unicode::whiteUpPointingTriangle;
+using WTF::Unicode::wordJoiner;
 using WTF::Unicode::yenSign;
 using WTF::Unicode::zeroWidthJoiner;
 using WTF::Unicode::zeroWidthNoBreakSpace;
index 0eb76b0..7d17e3d 100644 (file)
@@ -1,3 +1,16 @@
+2017-04-14  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Add performance test for asking the platform for a font for U+2060 WORD JOINER
+        https://bugs.webkit.org/show_bug.cgi?id=170842
+
+        Reviewed by Tim Horton.
+
+        No new tests because there is no behavior change.
+
+        * platform/graphics/WidthCache.h:
+        (WebCore::WidthCache::add):
+        (WebCore::WidthCache::addSlowCase):
+
 2017-04-14  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r215350.
index 4909409..b5ad561 100644 (file)
@@ -140,22 +140,22 @@ public:
             return nullptr;
         // The width cache is not really profitable unless we're doing expensive glyph transformations.
         if (!hasKerningOrLigatures)
-            return 0;
+            return nullptr;
         // Word spacing and letter spacing can change the width of a word.
         if (hasWordSpacingOrLetterSpacing)
-            return 0;
+            return nullptr;
         // Since this is just a width cache, we don't have enough information to satisfy glyph queries.
         if (glyphOverflow)
-            return 0;
+            return nullptr;
         // If we allow tabs and a tab occurs inside a word, the width of the word varies based on its position on the line.
         if (run.allowTabs())
-            return 0;
+            return nullptr;
         if (static_cast<unsigned>(run.length()) > SmallStringKey::capacity())
-            return 0;
+            return nullptr;
 
         if (m_countdown > 0) {
             --m_countdown;
-            return 0;
+            return nullptr;
         }
 
         return addSlowCase(run.text(), entry);
@@ -207,7 +207,7 @@ private:
         // No need to be fancy: we're just trying to avoid pathological growth.
         m_singleCharMap.clear();
         m_map.clear();
-        return 0;
+        return nullptr;
     }
 
     typedef HashMap<SmallStringKey, float, SmallStringKeyHash, SmallStringKeyHashTraits> Map;