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

Unified Diff: chrome/browser/ui/cocoa/web_intent_sheet_controller_unittest.mm

Issue 10825352: Change picker model API to allow less notification messages (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/web_intent_sheet_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/web_intent_sheet_controller_unittest.mm b/chrome/browser/ui/cocoa/web_intent_sheet_controller_unittest.mm
index bee8441bd0932b495cca6d6f0da2885b2b1122bc..7fe065f45f1402fc23de1ea0ede34ba2a3fc3088 100644
--- a/chrome/browser/ui/cocoa/web_intent_sheet_controller_unittest.mm
+++ b/chrome/browser/ui/cocoa/web_intent_sheet_controller_unittest.mm
@@ -111,8 +111,10 @@ TEST_F(WebIntentPickerSheetControllerTest, PopulatedRows) {
TEST_F(WebIntentPickerSheetControllerTest, SuggestionView) {
WebIntentPickerModel model;
-
- model.AddSuggestedExtension(string16(), string16(), 2.5);
+ std::vector<WebIntentPickerModel::SuggestedExtension> suggestions;
+ suggestions.push_back(WebIntentPickerModel::SuggestedExtension(
+ string16(), string16(), 2.5));
+ model.AddSuggestedExtensions(suggestions);
[controller_ performLayoutWithModel:&model];
// Get subviews.

Powered by Google App Engine
This is Rietveld 408576698