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

Unified Diff: ios/chrome/browser/ui/context_menu/context_menu_coordinator_unittest.mm

Issue 2154163002: Change ContextMenu to use AlertCoordinator (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@alertCoordinatorDelegate
Patch Set: Typo Created 4 years, 5 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
« no previous file with comments | « ios/chrome/browser/ui/context_menu/context_menu_coordinator.mm ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/ui/context_menu/context_menu_coordinator_unittest.mm
diff --git a/ios/chrome/browser/ui/context_menu/context_menu_coordinator_unittest.mm b/ios/chrome/browser/ui/context_menu/context_menu_coordinator_unittest.mm
index 412ce79c1fd21204b9c72dd69149efbbe312820d..b2913f678a1ce44d4ee6c1a2ac02a54ede210589 100644
--- a/ios/chrome/browser/ui/context_menu/context_menu_coordinator_unittest.mm
+++ b/ios/chrome/browser/ui/context_menu/context_menu_coordinator_unittest.mm
@@ -35,8 +35,8 @@ TEST_F(ContextMenuCoordinatorTest, ValidateIsVisible) {
params.location = CGPointZero;
params.view.reset([[view_controller_ view] retain]);
menu_coordinator_.reset([[ContextMenuCoordinator alloc]
- initWithViewController:view_controller_
- params:params]);
+ initWithBaseViewController:view_controller_
+ params:params]);
[menu_coordinator_ start];
EXPECT_TRUE([menu_coordinator_ isVisible]);
@@ -48,8 +48,8 @@ TEST_F(ContextMenuCoordinatorTest, ValidateDismissalOnStop) {
params.location = CGPointZero;
params.view.reset([[view_controller_ view] retain]);
menu_coordinator_.reset([[ContextMenuCoordinator alloc]
- initWithViewController:view_controller_
- params:params]);
+ initWithBaseViewController:view_controller_
+ params:params]);
[menu_coordinator_ start];
[menu_coordinator_ stop];
@@ -63,8 +63,8 @@ TEST_F(ContextMenuCoordinatorTest, ValidateDismissalOnDestroy) {
params.location = CGPointZero;
params.view.reset([[view_controller_ view] retain]);
menu_coordinator_.reset([[ContextMenuCoordinator alloc]
- initWithViewController:view_controller_
- params:params]);
+ initWithBaseViewController:view_controller_
+ params:params]);
[menu_coordinator_ start];
menu_coordinator_.reset();
@@ -78,8 +78,8 @@ TEST_F(ContextMenuCoordinatorTest, ValidateActions) {
params.location = CGPointZero;
params.view.reset([[view_controller_ view] retain]);
menu_coordinator_.reset([[ContextMenuCoordinator alloc]
- initWithViewController:view_controller_
- params:params]);
+ initWithBaseViewController:view_controller_
+ params:params]);
NSArray* menu_titles = @[ @"foo", @"bar" ];
for (NSString* title in menu_titles) {
@@ -114,8 +114,8 @@ TEST_F(ContextMenuCoordinatorTest, CancelButtonExists) {
params.location = CGPointZero;
params.view.reset([[view_controller_ view] retain]);
menu_coordinator_.reset([[ContextMenuCoordinator alloc]
- initWithViewController:view_controller_
- params:params]);
+ initWithBaseViewController:view_controller_
+ params:params]);
[menu_coordinator_ start];
@@ -140,8 +140,8 @@ TEST_F(ContextMenuCoordinatorTest, ValidateContextMenuParams) {
params.menu_title.reset(title);
params.view.reset([[view_controller_ view] retain]);
menu_coordinator_.reset([[ContextMenuCoordinator alloc]
- initWithViewController:view_controller_
- params:params]);
+ initWithBaseViewController:view_controller_
+ params:params]);
[menu_coordinator_ start];
EXPECT_TRUE([[view_controller_ presentedViewController]
« no previous file with comments | « ios/chrome/browser/ui/context_menu/context_menu_coordinator.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698