Attachment 362405 Details for Bug 194822 – Patch

[patch] Patch

bug-194822-20190219114103.patch (text/plain), 12.57 KB, created by Timothy Hatcher on 2019-02-19 11:41:04 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Timothy Hatcher

Created: 2019-02-19 11:41:04 PST

Size: 12.57 KB

patch

obsolete

>Subversion Revision: 241761
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index a6da53fe8e04ad79dd06c117c410a040b2b97e76..49a29837371028f77165647746fca7d695cfc1a7 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,22 @@
>+2019-02-19  Timothy Hatcher  <timothy@apple.com>
>+
>+        RenderThemeIOS should use RenderTheme's color cache instead of its own.
>+        https://bugs.webkit.org/show_bug.cgi?id=194822
>+        rdar://problem/48208296
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Uses existing tests.
>+
>+        * css/CSSValueKeywords.in:
>+        * platform/graphics/Color.h:
>+        * platform/graphics/cg/ColorCG.cpp:
>+        (WebCore::makeRGBAFromCGColor):
>+        (WebCore::Color::Color):
>+        * rendering/RenderThemeIOS.h:
>+        * rendering/RenderThemeIOS.mm:
>+        (WebCore::RenderThemeIOS::systemColor const):
>+
> 2019-02-19  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r241722.
>diff --git a/Source/WebCore/PAL/ChangeLog b/Source/WebCore/PAL/ChangeLog
>index 7e5c531eee9d9cfa3cf747d62d6759dc7f96742b..3b688e9464f5737a8747eac75977506841f7023e 100644
>--- a/Source/WebCore/PAL/ChangeLog
>+++ b/Source/WebCore/PAL/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-02-19  Timothy Hatcher  <timothy@apple.com>
>+
>+        RenderThemeIOS should use RenderTheme's color cache instead of its own.
>+        https://bugs.webkit.org/show_bug.cgi?id=194822
>+        rdar://problem/48208296
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * pal/ios/UIKitSoftLink.h:
>+        * pal/ios/UIKitSoftLink.mm:
>+
> 2019-02-16  Darin Adler  <darin@apple.com>
> 
>         Continue reducing use of String::format, now focusing on hex: "%p", "%x", etc.
>diff --git a/Source/WebCore/PAL/pal/ios/UIKitSoftLink.h b/Source/WebCore/PAL/pal/ios/UIKitSoftLink.h
>index f933959e62b8cce53cf13309b8496dea7eb0c482..90529f368b19176f00748de98865ba9638264e74 100644
>--- a/Source/WebCore/PAL/pal/ios/UIKitSoftLink.h
>+++ b/Source/WebCore/PAL/pal/ios/UIKitSoftLink.h
>@@ -56,6 +56,7 @@ SOFT_LINK_CLASS_FOR_HEADER(PAL, UILabel)
> SOFT_LINK_CLASS_FOR_HEADER(PAL, UIPasteboard)
> SOFT_LINK_CLASS_FOR_HEADER(PAL, UIScreen)
> SOFT_LINK_CLASS_FOR_HEADER(PAL, UITapGestureRecognizer)
>+SOFT_LINK_CLASS_FOR_HEADER(PAL, UITraitCollection)
> SOFT_LINK_CLASS_FOR_HEADER(PAL, UIView)
> SOFT_LINK_CLASS_FOR_HEADER(PAL, UIViewController)
> SOFT_LINK_CLASS_FOR_HEADER(PAL, UIWindow)
>diff --git a/Source/WebCore/PAL/pal/ios/UIKitSoftLink.mm b/Source/WebCore/PAL/pal/ios/UIKitSoftLink.mm
>index 33715ccc9bcd55dde959e2871d8b956eebb7516a..c2ed364c68d424a24cd987137aac33802679f949 100644
>--- a/Source/WebCore/PAL/pal/ios/UIKitSoftLink.mm
>+++ b/Source/WebCore/PAL/pal/ios/UIKitSoftLink.mm
>@@ -56,6 +56,7 @@ SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UILabel)
> SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UIPasteboard)
> SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UIScreen)
> SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UITapGestureRecognizer)
>+SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UITraitCollection)
> SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UIView)
> SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UIViewController)
> SOFT_LINK_CLASS_FOR_SOURCE(PAL, UIKit, UIWindow)
>diff --git a/Source/WebCore/css/CSSValueKeywords.in b/Source/WebCore/css/CSSValueKeywords.in
>index d8d3d2d544b787fc97c426073a62b9ca678ef4de..7e0e1706eebcf1ebfd24fdbdb43fa5a560c4935c 100644
>--- a/Source/WebCore/css/CSSValueKeywords.in
>+++ b/Source/WebCore/css/CSSValueKeywords.in
>@@ -222,22 +222,11 @@ threedshadow
> window
> windowframe
> windowtext
>-#if defined(WTF_PLATFORM_MAC) && WTF_PLATFORM_MAC
>+#if (defined(WTF_PLATFORM_MAC) && WTF_PLATFORM_MAC) || (defined(WTF_PLATFORM_IOS_FAMILY) && WTF_PLATFORM_IOS_FAMILY)
> -apple-system-header-text
> -apple-system-text-background
> -apple-system-control-background
>--apple-system-alternate-selected-text
>--apple-system-control-accent
>--apple-system-even-alternating-content-background
>--apple-system-odd-alternating-content-background
>--apple-system-selected-content-background
>--apple-system-unemphasized-selected-content-background
>--apple-system-selected-text
>--apple-system-unemphasized-selected-text
>--apple-system-selected-text-background
>--apple-system-unemphasized-selected-text-background
> -apple-system-placeholder-text
>--apple-system-find-highlight-background
> -apple-system-label
> -apple-system-secondary-label
> -apple-system-tertiary-label
>@@ -245,7 +234,14 @@ windowtext
> -apple-system-grid
> -apple-system-separator
> -apple-system-container-border
>-#endif
>+-apple-system-background
>+-apple-system-secondary-background
>+-apple-system-tertiary-background
>+-apple-system-grouped-background
>+-apple-system-secondary-grouped-background
>+-apple-system-tertiary-grouped-background
>+-apple-system-selected-content-background
>+-apple-system-unemphasized-selected-content-background
> -apple-wireless-playback-target-active
> -apple-system-blue
> -apple-system-brown
>@@ -256,6 +252,22 @@ windowtext
> -apple-system-purple
> -apple-system-red
> -apple-system-yellow
>+#endif
>+#if defined(WTF_PLATFORM_MAC) && WTF_PLATFORM_MAC
>+-apple-system-alternate-selected-text
>+-apple-system-control-accent
>+-apple-system-even-alternating-content-background
>+-apple-system-odd-alternating-content-background
>+-apple-system-selected-text
>+-apple-system-unemphasized-selected-text
>+-apple-system-selected-text-background
>+-apple-system-unemphasized-selected-text-background
>+-apple-system-find-highlight-background
>+#endif
>+#if defined(WTF_PLATFORM_IOS_FAMILY) && WTF_PLATFORM_IOS_FAMILY
>+-apple-system-indigo
>+-apple-system-teal
>+#endif
> -webkit-focus-ring-color
> currentcolor
> //
>diff --git a/Source/WebCore/platform/graphics/Color.h b/Source/WebCore/platform/graphics/Color.h
>index 5d039bbc20e6794f230e6140b409f338e59c71b1..d6915c0cc7a6ddcc64be2a7c79e2cda6c46d416e 100644
>--- a/Source/WebCore/platform/graphics/Color.h
>+++ b/Source/WebCore/platform/graphics/Color.h
>@@ -262,6 +262,7 @@ public:
> 
> #if USE(CG)
>     WEBCORE_EXPORT Color(CGColorRef);
>+    WEBCORE_EXPORT Color(CGColorRef, SemanticTag);
> #endif
> 
> #if PLATFORM(WIN)
>diff --git a/Source/WebCore/platform/graphics/cg/ColorCG.cpp b/Source/WebCore/platform/graphics/cg/ColorCG.cpp
>index f6e655a7553671bfd0241609b284327a69ea4cf5..109c640c52661a187ed510ad2f27000de53b7839 100644
>--- a/Source/WebCore/platform/graphics/cg/ColorCG.cpp
>+++ b/Source/WebCore/platform/graphics/cg/ColorCG.cpp
>@@ -51,13 +51,8 @@ RetainPtr<CGColorRef> TinyLRUCachePolicy<WebCore::Color, RetainPtr<CGColorRef>>:
> 
> namespace WebCore {
> 
>-Color::Color(CGColorRef color)
>+static RGBA32 makeRGBAFromCGColor(CGColorRef color)
> {
>-    if (!color) {
>-        m_colorData.rgbaAndFlags = invalidRGBAColor;
>-        return;
>-    }
>-
>     size_t numComponents = CGColorGetNumberOfComponents(color);
>     const CGFloat* components = CGColorGetComponents(color);
> 
>@@ -81,7 +76,29 @@ Color::Color(CGColorRef color)
>         ASSERT_NOT_REACHED();
>     }
> 
>-    setRGB(makeRGBA(r * 255, g * 255, b * 255, a * 255));
>+    static const double scaleFactor = nextafter(256.0, 0.0);
>+    return makeRGBA(r * scaleFactor, g * scaleFactor, b * scaleFactor, a * scaleFactor);
>+}
>+
>+Color::Color(CGColorRef color)
>+{
>+    if (!color) {
>+        m_colorData.rgbaAndFlags = invalidRGBAColor;
>+        return;
>+    }
>+
>+    setRGB(makeRGBAFromCGColor(color));
>+}
>+
>+Color::Color(CGColorRef color, SemanticTag)
>+{
>+    if (!color) {
>+        m_colorData.rgbaAndFlags = invalidRGBAColor;
>+        return;
>+    }
>+
>+    setRGB(makeRGBAFromCGColor(color));
>+    setIsSemantic();
> }
> 
> static CGColorRef leakCGColor(const Color& color)
>diff --git a/Source/WebCore/rendering/RenderThemeIOS.h b/Source/WebCore/rendering/RenderThemeIOS.h
>index ed8927fc4bdca0f53a8ccf49cf4466797c03adef..738591035efa74e165b0c957272ab7f6199ece01 100644
>--- a/Source/WebCore/rendering/RenderThemeIOS.h
>+++ b/Source/WebCore/rendering/RenderThemeIOS.h
>@@ -159,8 +159,6 @@ private:
>     String m_legacyMediaControlsStyleSheet;
>     String m_mediaControlsStyleSheet;
> 
>-    mutable HashMap<int, Color> m_systemColorCache;
>-
> #if USE(SYSTEM_PREVIEW)
>     RetainPtr<CIContext> m_ciContext;
> #if HAVE(IOSURFACE)
>diff --git a/Source/WebCore/rendering/RenderThemeIOS.mm b/Source/WebCore/rendering/RenderThemeIOS.mm
>index a28a4adc272f71c2b483f3094f9417cf71b67f0f..113743f4e290feb60354748a8b54c5963d66af1a 100644
>--- a/Source/WebCore/rendering/RenderThemeIOS.mm
>+++ b/Source/WebCore/rendering/RenderThemeIOS.mm
>@@ -75,9 +75,14 @@
> #import <pal/spi/cocoa/CoreTextSPI.h>
> #import <pal/spi/ios/UIKitSPI.h>
> #import <wtf/NeverDestroyed.h>
>+#import <wtf/ObjCRuntimeExtras.h>
> #import <wtf/RefPtr.h>
> #import <wtf/StdLibExtras.h>
> 
>+#if USE(APPLE_INTERNAL_SDK) && __has_include(<WebKitAdditions/RenderThemeIOSAdditions.mm>)
>+#include <WebKitAdditions/RenderThemeIOSAdditions.mm>
>+#endif
>+
> @interface WebCoreRenderThemeBundle : NSObject
> @end
> 
>@@ -1413,56 +1418,60 @@ String RenderThemeIOS::mediaControlsBase64StringForIconNameAndType(const String&
> 
> Color RenderThemeIOS::systemColor(CSSValueID cssValueID, OptionSet<StyleColor::Options> options) const
> {
>+    const bool useSystemAppearance = options.contains(StyleColor::Options::UseSystemAppearance);
>     const bool forVisitedLink = options.contains(StyleColor::Options::ForVisitedLink);
> 
>+    auto& cache = colorCache(options);
>+
>+#if USE(APPLE_INTERNAL_SDK) && __has_include(<WebKitAdditions/RenderThemeIOSSystemColorAdditions.mm>)
>+#include <WebKitAdditions/RenderThemeIOSSystemColorAdditions.mm>
>+#endif
>+
>     // The system color cache below can't handle visited links. The only color value
>     // that cares about visited links is CSSValueWebkitLink, so handle it here by
>     // calling through to RenderTheme's base implementation.
>     if (forVisitedLink && cssValueID == CSSValueWebkitLink)
>         return RenderTheme::systemColor(cssValueID, options);
> 
>+    ASSERT_UNUSED(useSystemAppearance, !useSystemAppearance);
>     ASSERT(!forVisitedLink);
> 
>-    auto addResult = m_systemColorCache.add(cssValueID, Color());
>-    if (!addResult.isNewEntry)
>-        return addResult.iterator->value;
>-
>-    Color color;
>-    switch (cssValueID) {
>-    case CSSValueAppleWirelessPlaybackTargetActive:
>-        color = [PAL::getUIColorClass() systemBlueColor].CGColor;
>-        break;
>-    case CSSValueAppleSystemBlue:
>-        color = [PAL::getUIColorClass() systemBlueColor].CGColor;
>-        break;
>-    case CSSValueAppleSystemGray:
>-        color = [PAL::getUIColorClass() systemGrayColor].CGColor;
>-        break;
>-    case CSSValueAppleSystemGreen:
>-        color = [PAL::getUIColorClass() systemGreenColor].CGColor;
>-        break;
>-    case CSSValueAppleSystemOrange:
>-        color = [PAL::getUIColorClass() systemOrangeColor].CGColor;
>-        break;
>-    case CSSValueAppleSystemPink:
>-        color = [PAL::getUIColorClass() systemPinkColor].CGColor;
>-        break;
>-    case CSSValueAppleSystemRed:
>-        color = [PAL::getUIColorClass() systemRedColor].CGColor;
>-        break;
>-    case CSSValueAppleSystemYellow:
>-        color = [PAL::getUIColorClass() systemYellowColor].CGColor;
>-        break;
>-    default:
>-        break;
>-    }
>+    return cache.systemStyleColors.ensure(cssValueID, [this, cssValueID, options] () -> Color {
>+        auto cssColorToSelector = [cssValueID] () -> SEL {
>+#if USE(APPLE_INTERNAL_SDK) && __has_include(<WebKitAdditions/RenderThemeIOSColorToSelectorAdditions.mm>)
>+#include <WebKitAdditions/RenderThemeIOSColorToSelectorAdditions.mm>
>+#endif
> 
>-    if (!color.isValid())
>-        color = RenderTheme::systemColor(cssValueID, options);
>+            switch (cssValueID) {
>+            case CSSValueAppleWirelessPlaybackTargetActive:
>+            case CSSValueAppleSystemBlue:
>+                return @selector(systemBlueColor);
>+            case CSSValueAppleSystemGray:
>+                return @selector(systemGrayColor);
>+            case CSSValueAppleSystemGreen:
>+                return @selector(systemGreenColor);
>+            case CSSValueAppleSystemOrange:
>+                return @selector(systemOrangeColor);
>+            case CSSValueAppleSystemPink:
>+                return @selector(systemPinkColor);
>+            case CSSValueAppleSystemPurple:
>+                return @selector(systemPurpleColor);
>+            case CSSValueAppleSystemRed:
>+                return @selector(systemRedColor);
>+            case CSSValueAppleSystemYellow:
>+                return @selector(systemYellowColor);
>+            default:
>+                return nullptr;
>+            }
>+        };
> 
>-    addResult.iterator->value = color;
>+        if (auto selector = cssColorToSelector()) {
>+            if (auto color = wtfObjCMsgSend<UIColor *>(PAL::getUIColorClass(), selector))
>+                return Color(color.CGColor, Color::Semantic);
>+        }
> 
>-    return addResult.iterator->value;
>+        return RenderTheme::systemColor(cssValueID, options);
>+    }).iterator->value;
> }
> 
> #if ENABLE(ATTACHMENT_ELEMENT)

Actions: View | Formatted Diff | Diff

Attachments on bug 194822: 362405 | 362407 | 362426 | 362433 | 362458 | 362477