Bug 1298209 - Remove unused mIsPerspectiveItem. r?mattwoodrow draft
authorMarkus Stange <mstange@themasta.com>
Tue, 23 Aug 2016 01:17:22 -0400
changeset 405735 0d9ed20b435e63b0ea1dd3109e75235e79acf0e9
parent 405734 095d60fc85444125bfdc69bc533e8076741a60b7
child 405736 7b725f378de5f3e0b002462df8249d891e1a0093
push id27567
push usermstange@themasta.com
push dateThu, 25 Aug 2016 23:19:20 +0000
reviewersmattwoodrow
bugs1298209
milestone51.0a1
Bug 1298209 - Remove unused mIsPerspectiveItem. r?mattwoodrow MozReview-Commit-ID: 7KPw4wnihFZ
layout/base/FrameLayerBuilder.cpp
--- a/layout/base/FrameLayerBuilder.cpp
+++ b/layout/base/FrameLayerBuilder.cpp
@@ -673,17 +673,16 @@ struct NewLayerEntry {
     : mAnimatedGeometryRoot(nullptr)
     , mScrollClip(nullptr)
     , mLayerContentsVisibleRect(0, 0, -1, -1)
     , mLayerState(LAYER_INACTIVE)
     , mHideAllLayersBelow(false)
     , mOpaqueForAnimatedGeometryRootParent(false)
     , mPropagateComponentAlphaFlattening(true)
     , mUntransformedVisibleRegion(false)
-    , mIsPerspectiveItem(false)
   {}
   // mLayer is null if the previous entry is for a PaintedLayer that hasn't
   // been optimized to some other form (yet).
   RefPtr<Layer> mLayer;
   AnimatedGeometryRoot* mAnimatedGeometryRoot;
   const DisplayItemScrollClip* mScrollClip;
   // If non-null, this ScrollMetadata is set to the be the first ScrollMetadata
   // on the layer.
@@ -709,17 +708,16 @@ struct NewLayerEntry {
   bool mOpaqueForAnimatedGeometryRootParent;
 
   // If true, then the content flags for this layer should contribute
   // to our decision to flatten component alpha layers, false otherwise.
   bool mPropagateComponentAlphaFlattening;
   // mVisibleRegion is relative to the associated frame before
   // transform.
   bool mUntransformedVisibleRegion;
-  bool mIsPerspectiveItem;
 };
 
 class PaintedLayerDataTree;
 
 /**
  * This is tree node type for PaintedLayerDataTree.
  * Each node corresponds to a different animated geometry root, and contains
  * a stack of PaintedLayerDatas, in bottom-to-top order.
@@ -4150,19 +4148,16 @@ ContainerState::ProcessDisplayItems(nsDi
       NS_ASSERTION(FindIndexOfLayerIn(mNewChildLayers, ownLayer) < 0,
                    "Layer already in list???");
 
       NewLayerEntry* newLayerEntry = mNewChildLayers.AppendElement();
       newLayerEntry->mLayer = ownLayer;
       newLayerEntry->mAnimatedGeometryRoot = animatedGeometryRoot;
       newLayerEntry->mScrollClip = agrScrollClip;
       newLayerEntry->mLayerState = layerState;
-      if (itemType == nsDisplayItem::TYPE_PERSPECTIVE) {
-        newLayerEntry->mIsPerspectiveItem = true;
-      }
 
       // Don't attempt to flatten compnent alpha layers that are within
       // a forced active layer, or an active transform;
       if (itemType == nsDisplayItem::TYPE_TRANSFORM ||
           layerState == LAYER_ACTIVE_FORCE) {
         newLayerEntry->mPropagateComponentAlphaFlattening = false;
       }
       // nsDisplayTransform::BuildLayer must set layerContentsVisibleRect.