Suggestions UI - favicon item

ios/chrome/browser/ui/suggestions/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/suggestions/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"
@@ skipping 56 matching lines @@
 #pragma mark - SuggestionsExpandableCellDelegate
 
 - (void)collapseCell:(UICollectionViewCell*)cell {
   [self expand:NO cell:cell];
 }
 
 - (void)expandCell:(UICollectionViewCell*)cell {
   [self expand:YES cell:cell];
 }
 
+#pragma mark - SuggestionsFaviconCellDelegate
+
+- (void)openFaviconAtIndexPath:(NSIndexPath*)indexPath {
+  [self.suggestionCommandHandler openFaviconAtIndexPath:indexPath];

[lpromero, 2017/01/18 09:34:39]

This is where I think the suggestionCommandHandler should only get an index
back, not an index path.

[gambard, 2017/01/18 12:37:58]

Done.

+}
+
 #pragma mark - SuggestionsItemActions
 
 - (void)addNewItem:(id)sender {
   [self.suggestionCommandHandler addEmptyItem];
 }
 
 #pragma mark - SuggestionsCollectionUpdater forwarding
 
 - (void)addTextItem:(NSString*)title
            subtitle:(NSString*)subtitle
@@ skipping 62 matching lines @@
 
     [UIView
         animateWithDuration:kAnimationDuration
                  animations:^{
                    [self.collectionView.collectionViewLayout invalidateLayout];
                  }];
   }
 }
 
 @end