Backed out changeset c044d8842861 (bug 1387467) for build bustage at FrameBuilder.cpp:154 (no member GetName). r=backout on a CLOSED TREE

gfx/2d/Logging.h

@@ -132,7 +132,6 @@ enum class LogReason : int {
   InvalidDrawTarget,
   NativeFontResourceNotFound,
   UnscaledFontNotFound,
-  InvalidLayerType,
   // End
   MustBeLessThanThis = 101,
 };

gfx/layers/mlgpu/FrameBuilder.cpp

@@ -148,13 +148,6 @@ FrameBuilder::ProcessContainerLayer(ContainerLayer* aContainer,
 {
   LayerMLGPU* layer = aContainer->AsHostLayer()->AsLayerMLGPU();
 
-  // Diagnostic information for bug 1387467.
-  if (!layer) {
-    gfxDevCrash(LogReason::InvalidLayerType) <<
-      "Layer type is invalid: " << aContainer->GetName();
-    return false;
-  }
-
   // We don't want to traverse containers twice, so we only traverse them if
   // they haven't been prepared yet.
   bool isFirstVisit = !layer->IsPrepared();
@@ -178,12 +171,6 @@ FrameBuilder::ProcessContainerLayer(ContainerLayer* aContainer,
   // RefLayers do not have intermediate surfaces so this is guaranteed
   // to be a full-fledged ContainerLayerMLGPU.
   ContainerLayerMLGPU* viewContainer = layer->AsContainerLayerMLGPU();
-  if (!viewContainer) {
-    gfxDevCrash(LogReason::InvalidLayerType) <<
-      "Container layer type is invalid: " << layer->GetName();
-    return false;
-  }
-
   if (isFirstVisit && !viewContainer->GetInvalidRect().IsEmpty()) {
     // The RenderView constructor automatically attaches itself to the parent.
     RefPtr<RenderViewMLGPU> view = new RenderViewMLGPU(this, viewContainer, aView);