From f682dfd45cbcd03482b3ac2136165a38a2342915 Mon Sep 17 00:00:00 2001
From: Ryan Nystrom <rnystrom@instagram.com>
Date: Thu, 3 Nov 2016 16:27:18 -0700
Subject: [PATCH] Remove layout experiment

Summary: Experiment results were neutral. Removing.

Reviewed By: jessesquires

Differential Revision: D4127172

fbshipit-source-id: 1110ea1e0c08ad74f63840465f12be5b6b059c15
---
 Source/IGListAdapterUpdater.m | 11 -----------
 Source/IGListExperiments.h    |  1 -
 2 files changed, 12 deletions(-)

diff --git a/Source/IGListAdapterUpdater.m b/Source/IGListAdapterUpdater.m
index 699b5123..c1e28276 100644
--- a/Source/IGListAdapterUpdater.m
+++ b/Source/IGListAdapterUpdater.m
@@ -208,17 +208,6 @@ static NSArray *objectsWithDuplicateIdentifiersRemoved(NSArray<id<IGListDiffable
 
     @try {
         [delegate listAdapterUpdater:self willPerformBatchUpdatesWithCollectionView:collectionView];
-
-        if (IGListExperimentEnabled(self.experiments, IGListExperimentLayoutBeforeUpdate)) {
-            /**
-             There are traces where UICollectionView throws "Invalid update: invalid number of items in section i..." where
-             logs show that the problem section has a different "before" item count than the assert claims. Our hunch is
-             that there is some state corruption from too many unanimated updates happening rapidly and layout state
-             becoming out of sync.
-             */
-            [collectionView layoutIfNeeded];
-        }
-
         if (animated) {
             [collectionView performBatchUpdates:updateBlock completion:completionBlock];
         } else {
diff --git a/Source/IGListExperiments.h b/Source/IGListExperiments.h
index fd85f2ec..277bb9df 100644
--- a/Source/IGListExperiments.h
+++ b/Source/IGListExperiments.h
@@ -15,7 +15,6 @@
  Bitmask-able options used for prerelease feature testing.
  */
 typedef NS_OPTIONS (NSUInteger, IGListExperiment) {
-    IGListExperimentLayoutBeforeUpdate = 1 << 0,
     IGListExperimentUICVReloadedInSetter = 1 << 1,
 };