Unreviewed, rolling out r190701.
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Oct 2015 22:00:46 +0000 (22:00 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Oct 2015 22:00:46 +0000 (22:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=149937

"It did not help, will try a full roll out instead" (Requested
by cdumez on #webkit).

Reverted changeset:

"Partial revert of r187626 as it caused a PLT regression"
https://bugs.webkit.org/show_bug.cgi?id=149898
http://trac.webkit.org/changeset/190701

Patch by Commit Queue <commit-queue@webkit.org> on 2015-10-08

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/FontCache.h

index 2c37b3e..d5c9418 100644 (file)
@@ -1,3 +1,17 @@
+2015-10-08  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r190701.
+        https://bugs.webkit.org/show_bug.cgi?id=149937
+
+        "It did not help, will try a full roll out instead" (Requested
+        by cdumez on #webkit).
+
+        Reverted changeset:
+
+        "Partial revert of r187626 as it caused a PLT regression"
+        https://bugs.webkit.org/show_bug.cgi?id=149898
+        http://trac.webkit.org/changeset/190701
+
 2015-10-08  Zalan Bujtas  <zalan@apple.com>
 
         Fallback to the RenderView when repaint container is null.
index 93ad089..9473ab1 100644 (file)
@@ -75,6 +75,7 @@ struct FontDescriptionKey {
         : m_size(description.computedPixelSize())
         , m_weight(description.weight())
         , m_flags(makeFlagsKey(description))
+        , m_locale(description.locale())
         , m_featureSettings(description.featureSettings())
     { }
 
@@ -84,7 +85,7 @@ struct FontDescriptionKey {
 
     bool operator==(const FontDescriptionKey& other) const
     {
-        return m_size == other.m_size && m_weight == other.m_weight && m_flags == other.m_flags
+        return m_size == other.m_size && m_weight == other.m_weight && m_flags == other.m_flags && m_locale == other.m_locale
             && m_featureSettings == other.m_featureSettings;
     }
 
@@ -102,6 +103,7 @@ struct FontDescriptionKey {
         hasher.add(m_weight);
         for (unsigned flagItem : m_flags)
             hasher.add(flagItem);
+        hasher.add(m_locale.isNull() ? 0 : m_locale.impl()->existingHash());
         hasher.add(m_featureSettings.hash());
         return hasher.hash();
     }
@@ -142,6 +144,7 @@ private:
     unsigned m_size { 0 };
     unsigned m_weight { 0 };
     std::array<unsigned, 2> m_flags {{ 0, 0 }};
+    AtomicString m_locale;
     FontFeatureSettings m_featureSettings;
 };