Bug 900745 - Remove bogus fallback codepath. r=mattwoodrow a=gchang
authorKartikaya Gupta <kgupta@mozilla.com>
Sat, 28 Jan 2017 09:06:31 -0500
changeset 378137 7ed6eee40e2995d597a912b8a7bf3f3459756d33
parent 378136 96f96e97c8614c96471631f502694027a48f40ae
child 378138 1634b7d9ef87d35a6eca538aec8fe1aaeeea3155
push id1419
push userjlund@mozilla.com
push dateMon, 10 Apr 2017 20:44:07 +0000
treeherdermozilla-release@5e6801b73ef6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmattwoodrow, gchang
bugs900745
milestone53.0a2
Bug 900745 - Remove bogus fallback codepath. r=mattwoodrow a=gchang MozReview-Commit-ID: GnGZFFRRTgE
gfx/layers/ipc/CrossProcessCompositorBridgeParent.cpp
--- a/gfx/layers/ipc/CrossProcessCompositorBridgeParent.cpp
+++ b/gfx/layers/ipc/CrossProcessCompositorBridgeParent.cpp
@@ -87,19 +87,17 @@ CrossProcessCompositorBridgeParent::Allo
     LayerTransactionParent* p = new LayerTransactionParent(lm, this, aId);
     p->AddIPDLReference();
     sIndirectLayerTrees[aId].mLayerTree = p;
     p->SetPendingCompositorUpdates(state->mPendingCompositorUpdates);
     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)
 {