Title: [161533] trunk/Source/WebKit2
Revision
161533
Author
[email protected]
Date
2014-01-08 18:12:17 -0800 (Wed, 08 Jan 2014)

Log Message

Make WK2 build when ENABLE(ASYNC_SCROLLING) is not defined
https://bugs.webkit.org/show_bug.cgi?id=126675

Reviewed by Tim Horton.

We need a stub RemoteScrollingCoordinatorTransaction outside
the #ifdef so that RemoteLayerTreeDrawingAreaProxy messages
work.

* Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:
(WebKit::RemoteScrollingCoordinatorTransaction::encode):
(WebKit::RemoteScrollingCoordinatorTransaction::decode):
* Shared/Scrolling/RemoteScrollingCoordinatorTransaction.h:
* UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm:
(WebKit::RemoteLayerTreeDrawingAreaProxy::commitLayerTree):
* WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:
(WebKit::RemoteLayerTreeDrawingArea::flushLayers):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (161532 => 161533)


--- trunk/Source/WebKit2/ChangeLog	2014-01-09 02:11:59 UTC (rev 161532)
+++ trunk/Source/WebKit2/ChangeLog	2014-01-09 02:12:17 UTC (rev 161533)
@@ -1,3 +1,23 @@
+2014-01-08  Simon Fraser  <[email protected]>
+
+        Make WK2 build when ENABLE(ASYNC_SCROLLING) is not defined
+        https://bugs.webkit.org/show_bug.cgi?id=126675
+
+        Reviewed by Tim Horton.
+        
+        We need a stub RemoteScrollingCoordinatorTransaction outside
+        the #ifdef so that RemoteLayerTreeDrawingAreaProxy messages
+        work.
+
+        * Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:
+        (WebKit::RemoteScrollingCoordinatorTransaction::encode):
+        (WebKit::RemoteScrollingCoordinatorTransaction::decode):
+        * Shared/Scrolling/RemoteScrollingCoordinatorTransaction.h:
+        * UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm:
+        (WebKit::RemoteLayerTreeDrawingAreaProxy::commitLayerTree):
+        * WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:
+        (WebKit::RemoteLayerTreeDrawingArea::flushLayers):
+
 2014-01-08  Tim Horton  <[email protected]>
 
         TileController can fail to receive exposedRect from the drawing area if set at the wrong time

Modified: trunk/Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp (161532 => 161533)


--- trunk/Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp	2014-01-09 02:11:59 UTC (rev 161532)
+++ trunk/Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp	2014-01-09 02:12:17 UTC (rev 161533)
@@ -26,8 +26,6 @@
 #include "config.h"
 #include "RemoteScrollingCoordinatorTransaction.h"
 
-#if ENABLE(ASYNC_SCROLLING)
-
 #include "ArgumentCoders.h"
 #include "MessageDecoder.h"
 #include "MessageEncoder.h"
@@ -41,6 +39,8 @@
 
 using namespace WebCore;
 
+#if ENABLE(ASYNC_SCROLLING)
+
 namespace IPC {
 
 template<> struct ArgumentCoder<ScrollingStateNode> {
@@ -304,4 +304,19 @@
 
 } // namespace WebKit
 
+#else // !ENABLE(ASYNC_SCROLLING)
+
+namespace WebKit {
+
+void RemoteScrollingCoordinatorTransaction::encode(IPC::ArgumentEncoder&) const
+{
+}
+
+bool RemoteScrollingCoordinatorTransaction::decode(IPC::ArgumentDecoder& decoder, RemoteScrollingCoordinatorTransaction& transaction)
+{
+    return true;
+}
+
+} // namespace WebKit
+
 #endif // ENABLE(ASYNC_SCROLLING)

Modified: trunk/Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.h (161532 => 161533)


--- trunk/Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.h	2014-01-09 02:11:59 UTC (rev 161532)
+++ trunk/Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.h	2014-01-09 02:12:17 UTC (rev 161533)
@@ -26,8 +26,6 @@
 #ifndef RemoteScrollingCoordinatorTransaction_h
 #define RemoteScrollingCoordinatorTransaction_h
 
-#if ENABLE(ASYNC_SCROLLING)
-
 #include <WebCore/ScrollingStateTree.h>
 #include <wtf/OwnPtr.h>
 #include <wtf/PassOwnPtr.h>
@@ -41,20 +39,23 @@
 
 class RemoteScrollingCoordinatorTransaction {
 public:
+
+#if ENABLE(ASYNC_SCROLLING)
     void setStateTreeToEncode(PassOwnPtr<WebCore::ScrollingStateTree> stateTree) { m_scrollingStateTree = stateTree; }
     OwnPtr<WebCore::ScrollingStateTree>& scrollingStateTree() { return m_scrollingStateTree; }
-    
+#endif // ENABLE(ASYNC_SCROLLING)
+
     void encode(IPC::ArgumentEncoder&) const;
     static bool decode(IPC::ArgumentDecoder&, RemoteScrollingCoordinatorTransaction&);
 
 private:
+#if ENABLE(ASYNC_SCROLLING)
     bool decode(IPC::ArgumentDecoder&);
     
     OwnPtr<WebCore::ScrollingStateTree> m_scrollingStateTree;
+#endif // ENABLE(ASYNC_SCROLLING)
 };
 
 } // namespace WebKit
 
-#endif // ENABLE(ASYNC_SCROLLING)
-
 #endif // RemoteScrollingCoordinatorTransaction_h

Modified: trunk/Source/WebKit2/UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm (161532 => 161533)


--- trunk/Source/WebKit2/UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm	2014-01-09 02:11:59 UTC (rev 161532)
+++ trunk/Source/WebKit2/UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm	2014-01-09 02:12:17 UTC (rev 161533)
@@ -86,7 +86,9 @@
 void RemoteLayerTreeDrawingAreaProxy::commitLayerTree(const RemoteLayerTreeTransaction& layerTreeTransaction, const RemoteScrollingCoordinatorTransaction& scrollingTreeTransaction)
 {
     m_remoteLayerTreeHost.updateLayerTree(layerTreeTransaction);
+#if ENABLE(ASYNC_SCROLLING)
     m_webPageProxy->scrollingCoordinatorProxy()->updateScrollingTree(scrollingTreeTransaction);
+#endif
 }
 
 } // namespace WebKit

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm (161532 => 161533)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm	2014-01-09 02:11:59 UTC (rev 161532)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm	2014-01-09 02:12:17 UTC (rev 161533)
@@ -300,9 +300,11 @@
     m_remoteLayerTreeContext->buildTransaction(layerTransaction, *m_rootLayer);
 
     RemoteScrollingCoordinatorTransaction scrollingTransaction;
+#if ENABLE(ASYNC_SCROLLING)
     if (m_webPage->scrollingCoordinator())
         toRemoteScrollingCoordinator(m_webPage->scrollingCoordinator())->buildTransaction(scrollingTransaction);
-    
+#endif
+
     m_webPage->send(Messages::RemoteLayerTreeDrawingAreaProxy::CommitLayerTree(layerTransaction, scrollingTransaction));
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to