Changeset 226215 in webkit


Ignore:
Timestamp:
Dec 20, 2017 9:39:31 PM (6 years ago)
Author:
rniwa@webkit.org
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):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r226213 r226215  
     12017-12-20  Ryosuke Niwa  <rniwa@webkit.org>
     2
     3        isSafari check should take Safari Technology Preview into account
     4        https://bugs.webkit.org/show_bug.cgi?id=181076
     5
     6        Reviewed by Alex Christensen.
     7
     8        Fixed isSafari() so that it returns true for Safari Technology Preview. This bug resulted in the custom pasteboard
     9        types not being enabled in Safari Technology Preview. In long term, we should eliminate these isSafari() checks.
     10
     11        * platform/cocoa/RuntimeApplicationChecksCocoa.mm:
     12        (WebCore::MacApplication::isSafari):
     13
    1142017-12-20  Wenson Hsieh  <wenson_hsieh@apple.com>
    215
  • trunk/Source/WebCore/platform/cocoa/RuntimeApplicationChecksCocoa.mm

    r225772 r226215  
    8383bool MacApplication::isSafari()
    8484{
    85     static bool isSafari = applicationBundleIsEqualTo("com.apple.Safari");
     85    static bool isSafari = applicationBundleIsEqualTo("com.apple.Safari") || applicationBundleIsEqualTo("com.apple.SafariTechnologyPreview");
    8686    return isSafari;
    8787}
Note: See TracChangeset for help on using the changeset viewer.