Connect ContentSuggestionsMediator to the ContentService

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

Index: ios/chrome/browser/ui/content_suggestions/content_suggestions_section_information.h
diff --git a/ios/chrome/browser/ui/content_suggestions/content_suggestions_section_information.h b/ios/chrome/browser/ui/content_suggestions/content_suggestions_section_information.h
index 92545c166d28a3203e860b8cff8e9d7457c6f312..8fff365f1cb350c4cf1823dec45e69f5f0cc233b 100644
--- a/ios/chrome/browser/ui/content_suggestions/content_suggestions_section_information.h
+++ b/ios/chrome/browser/ui/content_suggestions/content_suggestions_section_information.h
@@ -27,27 +27,25 @@ typedef NS_ENUM(NSInteger, ContentSuggestionsSectionID) {
 
   // Do not use this. It will trigger a DCHECK.
   // Do not add value after this one.
-  ContentSuggestionsSectionCount
+  ContentSuggestionsSectionUnknown
 };
 
 // Contains the information needed to display the section.
 @interface ContentSuggestionsSectionInformation : NSObject
 
 - (instancetype)initWithID:(ContentSuggestionsSectionID)ID
-                 emptyCell:(CollectionViewItem*)emptyCell
-                    layout:(ContentSuggestionsSectionLayout)layout
-                     title:(NSString*)title NS_DESIGNATED_INITIALIZER;
+    NS_DESIGNATED_INITIALIZER;
 - (instancetype)init NS_UNAVAILABLE;
 
 // Item to display when the section is empty. If nil the section should not be
 // displayed when empty.
-@property(nonatomic, strong, readonly) CollectionViewItem* emptyCell;
+@property(nonatomic, strong) CollectionViewItem* emptyCell;

[stkhapugin, 2017/02/15 15:35:43]

Why are these not readonly anymore? Can these be modified after creation? 

[gambard, 2017/02/16 10:06:30]

No, but the creation is in multiple steps (see ContentSuggestionsMediator's
SectionInformationFromCategoryInfo), it is easier to set the property this way.
And none of them are mandatory.

[stkhapugin, 2017/02/16 10:53:27]

So you agree that it's better and safer to make everything readonly, but you
won't do this because in one method where you instantiate this class you don't
want to create a few local variables? :)

 // Layout to display the content of the section.
-@property(nonatomic, assign, readonly) ContentSuggestionsSectionLayout layout;
+@property(nonatomic, assign) ContentSuggestionsSectionLayout layout;
 // ID of the section. Used for ordering.
 @property(nonatomic, assign, readonly) ContentSuggestionsSectionID ID;
 // Title for the section.
-@property(nonatomic, copy, readonly) NSString* title;
+@property(nonatomic, copy) NSString* title;
 
 @end