Open Most Visited items

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"
@@ skipping 123 matching lines @@
 
 - (void)collectionView:(UICollectionView*)collectionView
     didSelectItemAtIndexPath:(NSIndexPath*)indexPath {
   [super collectionView:collectionView didSelectItemAtIndexPath:indexPath];
 
   CollectionViewItem* item =
       [self.collectionViewModel itemAtIndexPath:indexPath];
   switch ([self.collectionUpdater contentSuggestionTypeForItem:item]) {
     case ContentSuggestionTypeReadingList:
     case ContentSuggestionTypeArticle:
+      [self blurOmnibox];
       [self.suggestionCommandHandler openPageForItem:item];
       break;
     case ContentSuggestionTypeMostVisited:
-      // TODO(crbug.com/707754): Open the most visited site.
+      [self blurOmnibox];
+      [self.suggestionCommandHandler openMostVisitedItem:item
+                                                 atIndex:indexPath.item];
       break;
     case ContentSuggestionTypeEmpty:
       break;
   }
 }
 
 #pragma mark - UICollectionViewDelegateFlowLayout
 
 - (CGSize)collectionView:(UICollectionView*)collectionView
                     layout:(UICollectionViewLayout*)collectionViewLayout
@@ skipping 106 matching lines @@
 // Must be called from inside a performBatchUpdates: block.
 - (void)addEmptySectionPlaceholderIfNeeded:(NSInteger)section {
   if ([self.collectionViewModel numberOfItemsInSection:section] > 0)
     return;
 
   NSIndexPath* emptyItem =
       [self.collectionUpdater addEmptyItemForSection:section];
   [self.collectionView insertItemsAtIndexPaths:@[ emptyItem ]];
 }
 
+// Tells WebToolbarController to resign focus to the omnibox.
+- (void)blurOmnibox {

[marq (ping after 24h), 2017/06/02 08:25:47]

Minor naming nit: when 'focus' is used to mean 'make the active UI element', I
don't think of 'blur' being the inverse operation; my expectation was that
'blurOmnibox' would apply a blur effect. 'defocusOmibox' would make more sense
to me. It's possible this is just me, though.

[gambard, 2017/06/02 13:26:58]

I had the same reaction as you :)
But as defocus was not in my dictionary I stuck with the method name currently
used in Google Landing Controller.
If you think "defocus" makes sense in english I would be glad to use it. I also
thought of "unfocus".
Do you have any preference?

+  // TODO(crbug.com/700375): once the omnibox is part of Content Suggestions,
+  // remove the fake omnibox focus here.
+}
+
 @end