Changeset 176693 in webkit

Timestamp:

Dec 2, 2014, 1:53:06 PM (12 years ago)

Author:

Beth Dakin

Message:

Yet another build fix.

• WebView/WebActionMenuController.mm:

(-[WebActionMenuController _defaultMenuItemsForLink]):

Location:

trunk/Source/WebKit/mac

Files:

• ChangeLog (modified) (1 diff)
• WebView/WebActionMenuController.mm (modified) (2 diffs)

trunk/Source/WebKit/mac/ChangeLog

@@ +1 @@
+2014-12-02  Beth Dakin  <bdakin@apple.com>
+
+        Yet another build fix.
+
+        * WebView/WebActionMenuController.mm:
+        (-[WebActionMenuController _defaultMenuItemsForLink]):
+
 2014-12-02  Beth Dakin  <bdakin@apple.com>
 

trunk/Source/WebKit/mac/WebView/WebActionMenuController.mm

@@ -282 +282 @@
     RetainPtr<NSMenuItem> openLinkItem = [self _createActionMenuItemForTag:WebActionMenuItemTagOpenLinkInDefaultBrowser];
 
-    RetainPtr<QLPreviewMenuItem> previewLinkItem;
-    if ([NSMenuItem respondsToSelector:@selector(standardQuickLookMenuItem)]) {
-        previewLinkItem = [NSMenuItem standardQuickLookMenuItem];
-        [previewLinkItem setPreviewStyle:QLPreviewStylePopover];
-        [previewLinkItem setDelegate:self];
+    BOOL shouldUseStandardQuickLookPreview = [NSMenuItem respondsToSelector:@selector(standardQuickLookMenuItem)];
+    RetainPtr<NSMenuItem> previewLinkItem;
+    RetainPtr<QLPreviewMenuItem> qlPreviewLinkItem;
+    if (shouldUseStandardQuickLookPreview) {
+        qlPreviewLinkItem = [NSMenuItem standardQuickLookMenuItem];
+        [qlPreviewLinkItem setPreviewStyle:QLPreviewStylePopover];
+        [qlPreviewLinkItem setDelegate:self];
     } else
         previewLinkItem = [NSMenuItem separatorItem];
@@ -292 +294 @@
     RetainPtr<NSMenuItem> readingListItem = [self _createActionMenuItemForTag:WebActionMenuItemTagAddLinkToSafariReadingList];
 
-    return @[ openLinkItem.get(), previewLinkItem.get(), [NSMenuItem separatorItem], readingListItem.get() ];
+    return @[ openLinkItem.get(), shouldUseStandardQuickLookPreview ? qlPreviewLinkItem.get() : previewLinkItem.get(), [NSMenuItem separatorItem], readingListItem.get() ];
 }