Backed out changeset 69bc1e04bb2c (bug 1283947) for build bustage. r=backout on a CLOSED TREE

gfx/layers/TransactionIdAllocator.h

@@ -53,15 +53,6 @@ public:
    */
   virtual void RevokeTransactionId(uint64_t aTransactionId) = 0;
 
-  /**
-   * Stop waiting for pending transactions, if any.
-   *
-   * This is used when ClientLayerManager is assigning to another refresh
-   * driver, and the current refresh driver may never receive transaction
-   * completed notifications.
-   */
-  virtual void ClearPendingTransactions() = 0;
-
   /**
    * Get the start time of the current refresh tick.
    */

gfx/layers/client/ClientLayerManager.cpp

@@ -509,19 +509,6 @@ ClientLayerManager::GetCompositorSideAPZTestData(APZTestData* aData) const
   }
 }
 
-void
-ClientLayerManager::SetTransactionIdAllocator(TransactionIdAllocator* aAllocator)
-{
-  // When changing the refresh driver, the previous refresh driver may never
-  // receive updates of pending transactions it's waiting for. So clear the
-  // waiting state before assigning another refresh driver.
-  if (mTransactionIdAllocator) {
-    mTransactionIdAllocator->ClearPendingTransactions();
-  }
-
-  mTransactionIdAllocator = aAllocator;
-}
-
 float
 ClientLayerManager::RequestProperty(const nsAString& aProperty)
 {

layout/base/nsRefreshDriver.cpp

@@ -2174,13 +2174,6 @@ nsRefreshDriver::RevokeTransactionId(uint64_t aTransactionId)
   mPendingTransaction--;
 }
 
-void
-nsRefreshDriver::ClearPendingTransactions()
-{
-  mCompletedTransaction = mPendingTransaction;
-  mWaitingForTransaction = false;
-}
-
 mozilla::TimeStamp
 nsRefreshDriver::GetTransactionStart()
 {

layout/base/nsRefreshDriver.h

@@ -319,7 +319,6 @@ public:
   uint64_t LastTransactionId() const override;
   void NotifyTransactionCompleted(uint64_t aTransactionId) override;
   void RevokeTransactionId(uint64_t aTransactionId) override;
-  void ClearPendingTransactions() override;
   mozilla::TimeStamp GetTransactionStart() override;
 
   bool IsWaitingForPaint(mozilla::TimeStamp aTime);