Remove duplicated temporary items loop
authorMatt Woodrow <mwoodrow@mozilla.com>
Fri, 15 Sep 2017 15:04:32 -0400
changeset 685639 4dbe9ae4248c0f5205342494ba4031539bc1e991
parent 685638 084f83687e37a34e8186ab2b7bbb6a20a7a0eb2f
child 685640 3e66c3bb56881547abf5c8e0fcc4190d13fa392a
push id86010
push userbmo:ethlin@mozilla.com
push dateWed, 25 Oct 2017 00:44:42 +0000
milestone57.0a1
Remove duplicated temporary items loop
layout/painting/nsDisplayList.cpp
--- a/layout/painting/nsDisplayList.cpp
+++ b/layout/painting/nsDisplayList.cpp
@@ -1194,19 +1194,16 @@ nsDisplayListBuilder::~nsDisplayListBuil
                "All frames should have been unmarked");
   NS_ASSERTION(mPresShellStates.Length() == 0,
                "All presshells should have been exited");
   NS_ASSERTION(!mCurrentTableItem, "No table item should be active");
 
   for (nsDisplayItem* i : mTemporaryItems) {
     i->Destroy(this);
   }
-  for (nsDisplayItem* i : mTemporaryItems) {
-    i->Destroy(this);
-  }
   for (DisplayItemClipChain* c : mClipChainsToDestroy) {
 #ifdef USE_HEAP_ALLOCATED_CLIP_CHAINS
     delete c;
 #else
      c->DisplayItemClipChain::~DisplayItemClipChain();
 #endif
   }