Title: [226215] trunk/Source/WebCore
Revision
226215
Author
rn...@webkit.org
Date
2017-12-20 21:39:31 -0800 (Wed, 20 Dec 2017)

Log Message

isSafari check should take Safari Technology Preview into account
https://bugs.webkit.org/show_bug.cgi?id=181076

Reviewed by Alex Christensen.

Fixed isSafari() so that it returns true for Safari Technology Preview. This bug resulted in the custom pasteboard
types not being enabled in Safari Technology Preview. In long term, we should eliminate these isSafari() checks.

* platform/cocoa/RuntimeApplicationChecksCocoa.mm:
(WebCore::MacApplication::isSafari):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (226214 => 226215)


--- trunk/Source/WebCore/ChangeLog	2017-12-21 04:37:55 UTC (rev 226214)
+++ trunk/Source/WebCore/ChangeLog	2017-12-21 05:39:31 UTC (rev 226215)
@@ -1,3 +1,16 @@
+2017-12-20  Ryosuke Niwa  <rn...@webkit.org>
+
+        isSafari check should take Safari Technology Preview into account
+        https://bugs.webkit.org/show_bug.cgi?id=181076
+
+        Reviewed by Alex Christensen.
+
+        Fixed isSafari() so that it returns true for Safari Technology Preview. This bug resulted in the custom pasteboard
+        types not being enabled in Safari Technology Preview. In long term, we should eliminate these isSafari() checks.
+
+        * platform/cocoa/RuntimeApplicationChecksCocoa.mm:
+        (WebCore::MacApplication::isSafari):
+
 2017-12-20  Wenson Hsieh  <wenson_hs...@apple.com>
 
         [Attachment Support] Attachment replacement logic should not depend on subresource URL attribute replacement

Modified: trunk/Source/WebCore/platform/cocoa/RuntimeApplicationChecksCocoa.mm (226214 => 226215)


--- trunk/Source/WebCore/platform/cocoa/RuntimeApplicationChecksCocoa.mm	2017-12-21 04:37:55 UTC (rev 226214)
+++ trunk/Source/WebCore/platform/cocoa/RuntimeApplicationChecksCocoa.mm	2017-12-21 05:39:31 UTC (rev 226215)
@@ -82,7 +82,7 @@
 
 bool MacApplication::isSafari()
 {
-    static bool isSafari = applicationBundleIsEqualTo("com.apple.Safari");
+    static bool isSafari = applicationBundleIsEqualTo("com.apple.Safari") || applicationBundleIsEqualTo("com.apple.SafariTechnologyPreview");
     return isSafari;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to