Unreviewed, rolling out r247462.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 16 Jul 2019 00:43:41 +0000 (00:43 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 16 Jul 2019 00:43:41 +0000 (00:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=199816

completely messed up the patch (Requested by litherum on
#webkit).

Reverted changeset:

"New York font erroneously gets synthetic bold"
https://bugs.webkit.org/show_bug.cgi?id=199653
https://trac.webkit.org/changeset/247462

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

12 files changed:
Source/WTF/ChangeLog
Source/WTF/wtf/Platform.h
Source/WebCore/ChangeLog
Source/WebCore/Modules/webgpu/WHLSL/WHLSLRecursionChecker.cpp
Source/WebCore/page/ProcessWarming.cpp
Source/WebCore/page/cocoa/MemoryReleaseCocoa.mm
Source/WebCore/platform/graphics/FontCascadeDescription.cpp
Source/WebCore/platform/graphics/FontDescription.h
Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp
Source/WebCore/platform/graphics/cocoa/FontDescriptionCocoa.cpp
Source/WebCore/platform/graphics/cocoa/FontFamilySpecificationCoreText.cpp
Source/WebCore/platform/graphics/cocoa/SystemFontDatabaseCoreText.cpp

index 53b603c..325488b 100644 (file)
@@ -1,3 +1,17 @@
+2019-07-15  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r247462.
+        https://bugs.webkit.org/show_bug.cgi?id=199816
+
+        completely messed up the patch (Requested by litherum on
+        #webkit).
+
+        Reverted changeset:
+
+        "New York font erroneously gets synthetic bold"
+        https://bugs.webkit.org/show_bug.cgi?id=199653
+        https://trac.webkit.org/changeset/247462
+
 2019-07-11  Myles C. Maxfield  <mmaxfield@apple.com>
 
         New York font erroneously gets synthetic bold
index 6b82053..5237af2 100644 (file)
 #define HAVE_SUBVIEWS_IVAR_SPI 1
 #endif
 
-#if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED >= 110000) || (PLATFORM(WATCHOS) && __WATCH_OS_VERSION_MIN_REQUIRED >= 40000) || (PLATFORM(APPLETV) && __TV_OS_VERSION_MIN_REQUIRED >= 110000)
-#define USE_PLATFORM_SYSTEM_FALLBACK_LIST 1
-#endif
-
 #if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101500) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED >= 130000) || (PLATFORM(WATCHOS) && __WATCH_OS_VERSION_MIN_REQUIRED >= 60000) || (PLATFORM(APPLETV) && __TV_OS_VERSION_MIN_REQUIRED >= 130000)
 #define HAVE_DESIGN_SYSTEM_UI_FONTS 1
 #endif
index e2a5d1f..e39c4d4 100644 (file)
@@ -1,3 +1,17 @@
+2019-07-15  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r247462.
+        https://bugs.webkit.org/show_bug.cgi?id=199816
+
+        completely messed up the patch (Requested by litherum on
+        #webkit).
+
+        Reverted changeset:
+
+        "New York font erroneously gets synthetic bold"
+        https://bugs.webkit.org/show_bug.cgi?id=199653
+        https://trac.webkit.org/changeset/247462
+
 2019-07-11  Simon Fraser  <simon.fraser@apple.com>
 
         Fix builds where HAVE_DESIGN_SYSTEM_UI_FONTS is not defined.
index ba497b3..6af88fe 100644 (file)
@@ -30,7 +30,6 @@
 
 #include "WHLSLCallExpression.h"
 #include "WHLSLFunctionDefinition.h"
-#include "WHLSLProgram.h"
 #include "WHLSLVisitor.h"
 #include <wtf/HashSet.h>
 
index 26aeac2..a158990 100644 (file)
@@ -72,7 +72,7 @@ void ProcessWarming::prewarmGlobally()
     // Prewarms JS VM.
     commonVM();
 
-#if USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#if USE_PLATFORM_SYSTEM_FALLBACK_LIST
     // Cache system UI font fallbacks. Almost every web process needs these.
     // Initializing one size is sufficient to warm CoreText caches.
     FontCascadeDescription systemFontDescription;
index d2121e7..2b8a3dc 100644 (file)
@@ -44,7 +44,7 @@ namespace WebCore {
 
 void platformReleaseMemory(Critical)
 {
-#if USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#if USE_PLATFORM_SYSTEM_FALLBACK_LIST
     SystemFontDatabaseCoreText::singleton().clear();
 #endif
     clearFontFamilySpecificationCoreTextCache();
index 8eb4471..34e5b8f 100644 (file)
@@ -62,7 +62,7 @@ FontCascadeDescription::FontCascadeDescription()
 {
 }
 
-#if !USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#if !USE_PLATFORM_SYSTEM_FALLBACK_LIST
 
 unsigned FontCascadeDescription::effectiveFamilyCount() const
 {
index c8d4a85..21de238 100644 (file)
@@ -31,6 +31,8 @@
 #include <unicode/uscript.h>
 #include <wtf/MathExtras.h>
 
+#define USE_PLATFORM_SYSTEM_FALLBACK_LIST ((PLATFORM(IOS_FAMILY) && __IPHONE_OS_VERSION_MIN_REQUIRED >= 110000) || (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300))
+
 namespace WebCore {
 
 using namespace WebKitFontFamilyNames;
index 062929c..80c1322 100644 (file)
@@ -1237,7 +1237,7 @@ static void invalidateFontCache()
         return;
     }
 
-#if USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#if USE_PLATFORM_SYSTEM_FALLBACK_LIST
     SystemFontDatabaseCoreText::singleton().clear();
 #endif
     clearFontFamilySpecificationCoreTextCache();
@@ -1393,7 +1393,7 @@ static RetainPtr<CTFontRef> lookupFallbackFont(CTFontRef font, FontSelectionValu
 
     CFIndex coveredLength = 0;
     RetainPtr<CTFontRef> result;
-#if !USE(PLATFORM_SYSTEM_FALLBACK_LIST) && (PLATFORM(MAC) || (PLATFORM(IOS_FAMILY) && TARGET_OS_IOS))
+#if !USE_PLATFORM_SYSTEM_FALLBACK_LIST && (PLATFORM(MAC) || (PLATFORM(IOS_FAMILY) && TARGET_OS_IOS))
     result = adoptCF(CTFontCreatePhysicalFontForCharactersWithLanguage(font, characters, length, localeString.get(), &coveredLength));
 #else
     result = adoptCF(CTFontCreateForCharactersWithLanguage(font, characters, length, localeString.get(), &coveredLength));
index 95577c2..43e6a93 100644 (file)
@@ -32,7 +32,7 @@
 
 namespace WebCore {
 
-#if USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#if USE_PLATFORM_SYSTEM_FALLBACK_LIST
 
 #if PLATFORM(IOS_FAMILY)
 
@@ -121,7 +121,7 @@ unsigned FontCascadeDescription::effectiveFamilyCount() const
 FontFamilySpecification FontCascadeDescription::effectiveFamilyAt(unsigned index) const
 {
     // The special cases in this function need to match the behavior in FontCacheIOS.mm and FontCacheMac.mm. On systems
-    // where USE(PLATFORM_SYSTEM_FALLBACK_LIST) is set to true, this code is used for regular (element style) lookups,
+    // where USE_PLATFORM_SYSTEM_FALLBACK_LIST is set to true, this code is used for regular (element style) lookups,
     // and the code in FontDescriptionCocoa.cpp is used when src:local(special-cased-name) is specified inside an
     // @font-face block.
     // FIXME: Currently, an @font-face block corresponds to a single item in the font-family: fallback list, which
@@ -144,7 +144,7 @@ FontFamilySpecification FontCascadeDescription::effectiveFamilyAt(unsigned index
     return nullAtom();
 }
 
-#endif // USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#endif // USE_PLATFORM_SYSTEM_FALLBACK_LIST
 
 static String computeSpecializedChineseLocale()
 {
index 1ce0c94..c96a2ed 100644 (file)
@@ -103,7 +103,7 @@ FontRanges FontFamilySpecificationCoreText::fontRanges(const FontDescription& fo
         auto font = adoptCF(CTFontCreateWithFontDescriptor(m_fontDescriptor.get(), size, nullptr));
 
         auto fontForSynthesisComputation = font;
-#if USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#if USE_PLATFORM_SYSTEM_FALLBACK_LIST
         if (auto physicalFont = adoptCF(CTFontCopyPhysicalFont(font.get())))
             fontForSynthesisComputation = physicalFont;
 #endif
index ac0e34b..3283280 100644 (file)
@@ -45,7 +45,7 @@ SystemFontDatabaseCoreText::SystemFontDatabaseCoreText()
 {
 }
 
-#if USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#if USE_PLATFORM_SYSTEM_FALLBACK_LIST
 
 RetainPtr<CTFontRef> SystemFontDatabaseCoreText::createSystemUIFont(const CascadeListParameters& parameters, CFStringRef locale)
 {
@@ -238,7 +238,7 @@ Vector<RetainPtr<CTFontDescriptorRef>> SystemFontDatabaseCoreText::cascadeList(c
     return cascadeList(systemFontParameters(description, cssFamily, clientUse, allowUserInstalledFonts), clientUse);
 }
 
-#endif // USE(PLATFORM_SYSTEM_FALLBACK_LIST)
+#endif // USE_PLATFORM_SYSTEM_FALLBACK_LIST
 
 static String genericFamily(const String& locale, HashMap<String, String>& map, CFStringRef ctKey)
 {