Suggested Articles can be dismissed

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

 // 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.
 
 #ifndef IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_VIEW_CONTROLLER_H_
 #define IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_VIEW_CONTROLLER_H_
 
 #import <UIKit/UIKit.h>
 
 #import "ios/chrome/browser/ui/collection_view/collection_view_controller.h"
@@ skipping 12 matching lines @@
                    dataSource:(id<ContentSuggestionsDataSource>)dataSource
     NS_DESIGNATED_INITIALIZER;
 
 - (instancetype)initWithStyle:(CollectionViewControllerStyle)style
     NS_UNAVAILABLE;
 
 // Handler for the commands sent by the ContentSuggestionsViewController.
 @property(nonatomic, weak) id<ContentSuggestionsCommands>
     suggestionCommandHandler;
 
+- (void)dismissEntryAtIndexPath:(NSIndexPath*)indexPath;

[lpromero, 2017/03/06 10:29:00]

Add a comment, so we know what is the contract: remove from collection view
only, or from view and CVModel?

[gambard, 2017/03/06 12:16:00]

Well, for me the fact that it has a CVModel is an implementation detail. But I
have added it in comment.

[lpromero, 2017/03/06 13:02:30]

True, too much details. As a rule of thumb, we should try to describe the
"high-level" flow instead of what it does underneath. Maybe the comment should
say something like: updates the view when the user wants to delete an entry.
Anyway, looks good as is.

[gambard, 2017/03/06 17:25:33]

Acknowledged.

+
 @end
 
 #endif  // IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_VIEW_CONTROLLER_H_