Revert "Ensure our transaction arrives at the compositor before asking the parent to paint"

This reverts commit e0ec2a72700f.

dom/base/nsDOMWindowUtils.cpp

@@ -338,7 +338,6 @@ nsDOMWindowUtils::UpdateLayerTree()
     if (view) {
       presShell->Paint(view, view->GetBounds(),
           nsIPresShell::PAINT_LAYERS | nsIPresShell::PAINT_SYNC_DECODE_IMAGES);
-      presShell->GetLayerManager()->WaitOnTransactionProcessed();
     }
   }
   return NS_OK;

gfx/layers/Layers.h

@@ -596,11 +596,6 @@ public:
    */
   virtual void FlushRendering() { }
 
-  /**
-   * Make sure that the previous transaction has been
-   * received. This will synchronsly wait on a remote compositor. */
-  virtual void WaitOnTransactionProcessed() { }
-
   virtual void SendInvalidRegion(const nsIntRegion& aRegion) {}
 
   /**

gfx/layers/client/ClientLayerManager.cpp

@@ -656,14 +656,6 @@ ClientLayerManager::FlushRendering()
   }
 }
 
-void
-ClientLayerManager::WaitOnTransactionProcessed()
-{
-  CompositorBridgeChild* remoteRenderer = GetCompositorBridgeChild();
-  if (remoteRenderer) {
-    remoteRenderer->SendWaitOnTransactionProcessed();
-  }
-}
 void
 ClientLayerManager::UpdateTextureFactoryIdentifier(const TextureFactoryIdentifier& aNewIdentifier,
                                                    uint64_t aDeviceResetSeqNo)

gfx/layers/client/ClientLayerManager.h

@@ -114,7 +114,6 @@ public:
   }
 
   virtual void FlushRendering() override;
-  virtual void WaitOnTransactionProcessed() override;
   virtual void SendInvalidRegion(const nsIntRegion& aRegion) override;
 
   virtual uint32_t StartFrameTimeRecording(int32_t aBufferSize) override;

gfx/layers/ipc/CompositorBridgeParent.cpp

@@ -523,12 +523,6 @@ CompositorBridgeParent::RecvMakeSnapshot(const SurfaceDescriptor& aInSnapshot,
   return IPC_OK();
 }
 
-mozilla::ipc::IPCResult
-CompositorBridgeParent::RecvWaitOnTransactionProcessed()
-{
-  return IPC_OK();
-}
-
 mozilla::ipc::IPCResult
 CompositorBridgeParent::RecvFlushRendering()
 {

gfx/layers/ipc/CompositorBridgeParent.h

@@ -199,7 +199,6 @@ public:
                                 const gfx::IntRect& aRect) override;
   virtual mozilla::ipc::IPCResult RecvFlushRendering() override;
   virtual mozilla::ipc::IPCResult RecvFlushRenderingAsync() override;
-  virtual mozilla::ipc::IPCResult RecvWaitOnTransactionProcessed() override;
   virtual mozilla::ipc::IPCResult RecvForcePresent() override;
 
   virtual mozilla::ipc::IPCResult RecvNotifyRegionInvalidated(const nsIntRegion& aRegion) override;

gfx/layers/ipc/CrossProcessCompositorBridgeParent.h

@@ -60,7 +60,6 @@ public:
   virtual mozilla::ipc::IPCResult RecvFlushRendering() override { return IPC_OK(); }
   virtual mozilla::ipc::IPCResult RecvFlushRenderingAsync() override { return IPC_OK(); }
   virtual mozilla::ipc::IPCResult RecvForcePresent() override { return IPC_OK(); }
-  virtual mozilla::ipc::IPCResult RecvWaitOnTransactionProcessed() override { return IPC_OK(); }
   virtual mozilla::ipc::IPCResult RecvNotifyRegionInvalidated(const nsIntRegion& aRegion) override { return IPC_OK(); }
   virtual mozilla::ipc::IPCResult RecvStartFrameTimeRecording(const int32_t& aBufferSize, uint32_t* aOutStartIndex) override { return IPC_OK(); }
   virtual mozilla::ipc::IPCResult RecvStopFrameTimeRecording(const uint32_t& aStartIndex, InfallibleTArray<float>* intervals) override  { return IPC_OK(); }

gfx/layers/ipc/PCompositorBridge.ipdl

@@ -194,9 +194,6 @@ parent:
   // synchronous repaints on resize.
   async FlushRenderingAsync();
 
-  // Make sure any pending composites have been received.
-  sync WaitOnTransactionProcessed();
-
   // Force an additional frame presentation to be executed. This is used to
   // work around a windows presentation bug (See Bug 1232042)
   async ForcePresent();

gfx/layers/wr/WebRenderLayerManager.cpp

@@ -107,7 +107,7 @@ WebRenderLayerManager::~WebRenderLayerManager()
 CompositorBridgeChild*
 WebRenderLayerManager::GetCompositorBridgeChild()
 {
-  return WrBridge()->GetCompositorBridgeChild();
+  return mWidget ? mWidget->GetRemoteRenderer() : nullptr;
 }
 
 int32_t
@@ -472,15 +472,6 @@ WebRenderLayerManager::FlushRendering()
   }
 }
 
-void
-WebRenderLayerManager::WaitOnTransactionProcessed()
-{
-  CompositorBridgeChild* bridge = GetCompositorBridgeChild();
-  if (bridge) {
-    bridge->SendWaitOnTransactionProcessed();
-  }
-}
-
 void
 WebRenderLayerManager::SendInvalidRegion(const nsIntRegion& aRegion)
 {

gfx/layers/wr/WebRenderLayerManager.h

@@ -88,7 +88,6 @@ public:
   virtual void RemoveDidCompositeObserver(DidCompositeObserver* aObserver) override;
 
   virtual void FlushRendering() override;
-  virtual void WaitOnTransactionProcessed() override;
 
   virtual void SendInvalidRegion(const nsIntRegion& aRegion) override;
 

ipc/ipdl/sync-messages.ini

@@ -995,8 +995,6 @@ description =
 description =
 [PCompositorBridge::FlushRendering]
 description =
-[PCompositorBridge::WaitOnTransactionProcessed]
-description = bug 1364626
 [PCompositorBridge::StartFrameTimeRecording]
 description =
 [PCompositorBridge::StopFrameTimeRecording]