Fix incorrect guards around a method declaration in PageClient.h
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Jul 2018 05:39:49 +0000 (05:39 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Jul 2018 05:39:49 +0000 (05:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=188153

Reviewed by Tim Horton.

`WebCore::DragItem` is forward declared under `ENABLE(DRAG_SUPPORT)`, but `startDrag` is declared under
`PLATFORM(COCOA)`. If `ENABLE(DRAG_SUPPORT)` is off but `PLATFORM(COCOA)` is on, the build breaks.

We fix this by moving `startDrag` under `ENABLE(DRAG_SUPPORT)`.

* UIProcess/PageClient.h:
(WebKit::PageClient::startDrag):

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/PageClient.h

index ec9cacb..f051f41 100644 (file)
@@ -1,3 +1,18 @@
+2018-07-29  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        Fix incorrect guards around a method declaration in PageClient.h
+        https://bugs.webkit.org/show_bug.cgi?id=188153
+
+        Reviewed by Tim Horton.
+
+        `WebCore::DragItem` is forward declared under `ENABLE(DRAG_SUPPORT)`, but `startDrag` is declared under
+        `PLATFORM(COCOA)`. If `ENABLE(DRAG_SUPPORT)` is off but `PLATFORM(COCOA)` is on, the build breaks.
+
+        We fix this by moving `startDrag` under `ENABLE(DRAG_SUPPORT)`.
+
+        * UIProcess/PageClient.h:
+        (WebKit::PageClient::startDrag):
+
 2018-07-28  Andy Estes  <aestes@apple.com>
 
         [Wi-Fi Assertions] Pass a ResumptionReason to resumeWiFiAssertions
index 52c8b9d..7ea82ba 100644 (file)
@@ -203,9 +203,13 @@ public:
 
     virtual void didChangeContentSize(const WebCore::IntSize&) = 0;
 
-#if PLATFORM(GTK) && ENABLE(DRAG_SUPPORT)
+#if ENABLE(DRAG_SUPPORT)
+#if PLATFORM(GTK)
     virtual void startDrag(Ref<WebCore::SelectionData>&&, WebCore::DragOperation, RefPtr<ShareableBitmap>&& dragImage) = 0;
+#else
+    virtual void startDrag(const WebCore::DragItem&, const ShareableBitmap::Handle&) { }
 #endif
+#endif // ENABLE(DRAG_SUPPORT)
 
     virtual void setCursor(const WebCore::Cursor&) = 0;
     virtual void setCursorHiddenUntilMouseMoves(bool) = 0;
@@ -219,7 +223,6 @@ public:
 #if PLATFORM(COCOA)
     virtual void accessibilityWebProcessTokenReceived(const IPC::DataReference&) = 0;
     virtual bool executeSavedCommandBySelector(const String& selector) = 0;
-    virtual void startDrag(const WebCore::DragItem&, const ShareableBitmap::Handle&) { }
     virtual void updateSecureInputState() = 0;
     virtual void resetSecureInputState() = 0;
     virtual void notifyInputContextAboutDiscardedComposition() = 0;