Disable QuickLook on tvOS
authoraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Sep 2015 07:33:05 +0000 (07:33 +0000)
committeraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Sep 2015 07:33:05 +0000 (07:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=149492
<rdar://problem/22741586>

Reviewed by Dan Bernstein.

Source/WebKit/mac:

We can't use USE(QUICK_LOOK) in a Private header, and postprocess-headers.sh doesn't know how to strip things
that aren't feature flags. So, if QuickLook is disabled, we'll just always define WebQuickLookFileNameKey and
WebQuickLookUTIKey, and we'll also define -quickLookContentForURL: but have it return nil.

* WebView/WebView.mm:
(-[WebView quickLookContentForURL:]):

Source/WTF:

* wtf/Platform.h: Stopped enabling QuickLook on tvOS.

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

Source/WTF/ChangeLog
Source/WTF/wtf/Platform.h
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/WebView/WebView.mm

index c632b8d..fef9f9c 100644 (file)
@@ -1,3 +1,13 @@
+2015-09-22  Andy Estes  <aestes@apple.com>
+
+        Disable QuickLook on tvOS
+        https://bugs.webkit.org/show_bug.cgi?id=149492
+        <rdar://problem/22741586>
+
+        Reviewed by Dan Bernstein.
+
+        * wtf/Platform.h: Stopped enabling QuickLook on tvOS.
+
 2015-09-22  Filip Pizlo  <fpizlo@apple.com>
 
         Get rid of ENABLE(PARALLEL_GC)
index 46d29fb..8723050 100644 (file)
 #endif
 #define USE_UIKIT_EDITING 1
 #define USE_WEB_THREAD 1
+
+#if !PLATFORM(APPLETV)
 #define USE_QUICK_LOOK 1
+#endif
 
 #if defined(TARGET_OS_IOS) && TARGET_OS_IOS && __IPHONE_OS_VERSION_MIN_REQUIRED >= 90000
 #define HAVE_APP_LINKS 1
index 2a86acd..632e4f7 100644 (file)
@@ -1,3 +1,18 @@
+2015-09-22  Andy Estes  <aestes@apple.com>
+
+        Disable QuickLook on tvOS
+        https://bugs.webkit.org/show_bug.cgi?id=149492
+        <rdar://problem/22741586>
+
+        Reviewed by Dan Bernstein.
+
+        We can't use USE(QUICK_LOOK) in a Private header, and postprocess-headers.sh doesn't know how to strip things
+        that aren't feature flags. So, if QuickLook is disabled, we'll just always define WebQuickLookFileNameKey and
+        WebQuickLookUTIKey, and we'll also define -quickLookContentForURL: but have it return nil.
+
+        * WebView/WebView.mm:
+        (-[WebView quickLookContentForURL:]):
+
 2015-09-22  Tim Horton  <timothy_horton@apple.com>
 
         Make it more obvious when using an unaccelerated image buffer, and fix a few callers who do
index e9fc862..9cb8a47 100644 (file)
@@ -611,7 +611,7 @@ NSString *_WebMainFrameURLKey =         @"mainFrameURL";
 NSString *_WebMainFrameDocumentKey =    @"mainFrameDocument";
 #endif
 
-#if USE(QUICK_LOOK)
+#if PLATFORM(IOS)
 NSString *WebQuickLookFileNameKey = @"WebQuickLookFileNameKey";
 NSString *WebQuickLookUTIKey      = @"WebQuickLookUTIKey";
 #endif
@@ -3570,9 +3570,10 @@ static inline IMP getMethod(id o, SEL s)
     return _private->page->setDefersLoading(defer);
 }
 
-#if TARGET_OS_IPHONE && USE(QUICK_LOOK)
+#if PLATFORM(IOS)
 - (NSDictionary *)quickLookContentForURL:(NSURL *)url
 {
+#if USE(QUICK_LOOK)
     NSString *uti = qlPreviewConverterUTIForURL(url);
     if (!uti)
         return nil;
@@ -3582,10 +3583,11 @@ static inline IMP getMethod(id o, SEL s)
         return nil;
 
     return [NSDictionary dictionaryWithObjectsAndKeys: fileName, WebQuickLookFileNameKey, uti, WebQuickLookUTIKey, nil];
-}
+#else
+    return nil;
 #endif
+}
 
-#if PLATFORM(IOS)
 - (BOOL)_isStopping
 {
     return _private->isStopping;