ContentSuggestions collection adds items with animation

ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h

Index: ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h
diff --git a/ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h b/ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h
index f1c79f69f4e905c7613353772a9451d8ad3d5361..e5f5ad2856330ae8f6ab9e3271de41e0d7b370f9 100644
--- a/ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h
+++ b/ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h
@@ -34,6 +34,14 @@
 // Returns the ContentSuggestionType associated with this item.
 - (ContentSuggestionType)contentSuggestionTypeForItem:(CollectionViewItem*)item;
 
+// Adds the |suggestions| to the model and returns their index paths.
+- (NSArray<NSIndexPath*>*)addSuggestionsToModel:

[lpromero, 2017/03/14 16:23:01]

Either remove "toModel" here or add it to the second method. Would it make sense
to move the second method above this one?

If someone calls -addSuggestionsToModel:, does it create the sections if they
don't exist? Or does one ensures first to call -addSectionsForSuggestions:?

[gambard, 2017/03/15 10:03:41]

You should call -addSectionForSuggestions: first. I have added a comment.
Added ToModel to the method name.

+    (NSArray<ContentSuggestion*>*)suggestions;
+
+// Adds the sections for the |suggestions| to the model and returns their index.
+- (NSIndexSet*)addSectionsForSuggestions:
+    (NSArray<ContentSuggestion*>*)suggestions;
+
 @end
 
 #endif  // IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_COLLECTION_UPDATER_H_