Layout Most Visited for ContentSuggestions

ios/chrome/browser/ui/content_suggestions/content_suggestions_view_controller.mm

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "ios/chrome/browser/ui/content_suggestions/content_suggestions_view_controller.h"
 
+#include "base/mac/foundation_util.h"
 #import "ios/chrome/browser/ui/collection_view/cells/MDCCollectionViewCell+Chrome.h"
 #import "ios/chrome/browser/ui/collection_view/cells/collection_view_item.h"
 #import "ios/chrome/browser/ui/collection_view/collection_view_model.h"
+#import "ios/chrome/browser/ui/content_suggestions/cells/content_suggestions_most_visited_cell.h"
 #import "ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h"
+#import "ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_utils.h"
 #import "ios/chrome/browser/ui/content_suggestions/content_suggestions_commands.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 namespace {
 using CSCollectionViewItem = CollectionViewItem<SuggestedContent>;
 }
 
@@ skipping 114 matching lines @@
       [self.suggestionCommandHandler openPageForItem:item];
       break;
     case ContentSuggestionTypeMostVisited:
       // TODO(crbug.com/707754): Open the most visited site.
       break;
     case ContentSuggestionTypeEmpty:
       break;
   }
 }
 
+#pragma mark - UICollectionViewDelegateFlowLayout
+
+- (CGSize)collectionView:(UICollectionView*)collectionView
+                    layout:(UICollectionViewLayout*)collectionViewLayout
+    sizeForItemAtIndexPath:(NSIndexPath*)indexPath {
+  if ([self.collectionUpdater isMostVisitedSection:indexPath.section])

[lody, 2017/05/17 08:23:33]

It looks like single-line ifs in this file use { } 

[gambard, 2017/05/17 09:39:43]

Done.

+    return [ContentSuggestionsMostVisitedCell defaultSize];
+  return [super collectionView:collectionView
+                        layout:collectionViewLayout
+        sizeForItemAtIndexPath:indexPath];
+}
+
+- (UIEdgeInsets)collectionView:(UICollectionView*)collectionView
+                        layout:(UICollectionViewLayout*)collectionViewLayout
+        insetForSectionAtIndex:(NSInteger)section {
+  if ([self.collectionUpdater isMostVisitedSection:section]) {
+    CGFloat margin = content_suggestions::centeredTilesMarginForWidth(
+        collectionView.frame.size.width);
+    return UIEdgeInsetsMake(0, margin, 0, margin);
+  }
+  return [super collectionView:collectionView
+                        layout:collectionViewLayout
+        insetForSectionAtIndex:section];
+}
+
+- (CGFloat)collectionView:(UICollectionView*)collectionView
+                                 layout:(UICollectionViewLayout*)
+                                            collectionViewLayout
+    minimumLineSpacingForSectionAtIndex:(NSInteger)section {
+  if ([self.collectionUpdater isMostVisitedSection:section]) {
+    return content_suggestions::spacingBetweenTiles();
+  }
+  return [super collectionView:collectionView
+                                   layout:collectionViewLayout
+      minimumLineSpacingForSectionAtIndex:section];
+}
+
 #pragma mark - MDCCollectionViewStylingDelegate
 
 - (UIColor*)collectionView:(nonnull UICollectionView*)collectionView
     cellBackgroundColorAtIndexPath:(nonnull NSIndexPath*)indexPath {
   if ([self.collectionUpdater
           shouldUseCustomStyleForSection:indexPath.section]) {
     return [UIColor clearColor];
   }
   return [UIColor whiteColor];
 }
@@ skipping 63 matching lines @@
 - (void)addEmptySectionPlaceholderIfNeeded:(NSInteger)section {
   if ([self.collectionViewModel numberOfItemsInSection:section] > 0)
     return;
 
   NSIndexPath* emptyItem =
       [self.collectionUpdater addEmptyItemForSection:section];
   [self.collectionView insertItemsAtIndexPaths:@[ emptyItem ]];
 }
 
 @end