Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(412)

Side by Side Diff: ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.h

Issue 2775593002: Add a message to empty ContentSuggestions sections (Closed)
Patch Set: Reviewable Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_COLLECTION _UPDATER_H_ 5 #ifndef IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_COLLECTION _UPDATER_H_
6 #define IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_COLLECTION _UPDATER_H_ 6 #define IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_COLLECTION _UPDATER_H_
7 7
8 #import <UIKit/UIKit.h> 8 #import <UIKit/UIKit.h>
9 9
10 #import "ios/chrome/browser/ui/content_suggestions/content_suggestion.h" 10 #import "ios/chrome/browser/ui/content_suggestions/content_suggestion.h"
(...skipping 23 matching lines...) Expand all
34 // Returns the ContentSuggestionType associated with this item. 34 // Returns the ContentSuggestionType associated with this item.
35 - (ContentSuggestionType)contentSuggestionTypeForItem:(CollectionViewItem*)item; 35 - (ContentSuggestionType)contentSuggestionTypeForItem:(CollectionViewItem*)item;
36 36
37 // Adds the sections for the |suggestions| to the model and returns their 37 // Adds the sections for the |suggestions| to the model and returns their
38 // indices. 38 // indices.
39 - (NSIndexSet*)addSectionsForSuggestionsToModel: 39 - (NSIndexSet*)addSectionsForSuggestionsToModel:
40 (NSArray<ContentSuggestion*>*)suggestions; 40 (NSArray<ContentSuggestion*>*)suggestions;
41 41
42 // Adds the |suggestions| to the model and returns their index paths. 42 // Adds the |suggestions| to the model and returns their index paths.
43 // The caller must ensure the corresponding sections have been added to the 43 // The caller must ensure the corresponding sections have been added to the
44 // model. 44 // model.
lpromero 2017/03/23 10:54:37 Maybe mention that if a section has no items, the
gambard 2017/03/23 15:25:49 The empty suggestion is a type of suggestion, with
45 - (NSArray<NSIndexPath*>*)addSuggestionsToModel: 45 - (NSArray<NSIndexPath*>*)addSuggestionsToModel:
46 (NSArray<ContentSuggestion*>*)suggestions; 46 (NSArray<ContentSuggestion*>*)suggestions;
47 47
48 // Adds the empty item for this section.
lpromero 2017/03/23 10:54:37 s/for/to? If you change, change also the method na
gambard 2017/03/23 15:25:49 Done.
49 - (void)addEmptyItemForSection:(NSInteger)section;
lpromero 2017/03/23 10:54:37 Please add unittests.
gambard 2017/03/23 15:25:49 Done.
50
48 @end 51 @end
49 52
50 #endif // IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_COLLECT ION_UPDATER_H_ 53 #endif // IOS_CHROME_BROWSER_UI_CONTENT_SUGGESTIONS_CONTENT_SUGGESTIONS_COLLECT ION_UPDATER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698