Bug 900745 - Remove bogus fallback codepath. r?mattwoodrow draft
authorKartikaya Gupta <kgupta@mozilla.com>
Sat, 28 Jan 2017 09:06:31 -0500
changeset 467662 951cf0d65c0377ce86efeeb0d513025fe69852ca
parent 467601 045d8fe30f546ab08466c9586ce298e6459c2069
child 543746 175269ed1c6a9bb0e7ae5eb2aef8835865e5ebbd
push id43241
push userkgupta@mozilla.com
push dateSat, 28 Jan 2017 14:06:57 +0000
reviewersmattwoodrow
bugs900745
milestone54.0a1
Bug 900745 - Remove bogus fallback codepath. r?mattwoodrow MozReview-Commit-ID: GnGZFFRRTgE
gfx/layers/ipc/CrossProcessCompositorBridgeParent.cpp
--- a/gfx/layers/ipc/CrossProcessCompositorBridgeParent.cpp
+++ b/gfx/layers/ipc/CrossProcessCompositorBridgeParent.cpp
@@ -89,19 +89,17 @@ CrossProcessCompositorBridgeParent::Allo
     sIndirectLayerTrees[aId].mLayerTree = p;
     if (state->mPendingCompositorUpdate) {
       p->SetPendingCompositorUpdate(state->mPendingCompositorUpdate.value());
     }
     return p;
   }
 
   NS_WARNING("Created child without a matching parent?");
-  // XXX: should be false, but that causes us to fail some tests on Mac w/ OMTC.
-  // Bug 900745. change *aSuccess to false to see test failures.
-  *aSuccess = true;
+  *aSuccess = false;
   LayerTransactionParent* p = new LayerTransactionParent(nullptr, this, aId);
   p->AddIPDLReference();
   return p;
 }
 
 bool
 CrossProcessCompositorBridgeParent::DeallocPLayerTransactionParent(PLayerTransactionParent* aLayers)
 {