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

Unified Diff: ui/base/ios/cru_context_menu_controller_unittest.mm

Issue 1103743002: Use UIAlertController on iOS8 for Context Menu. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resolved Lane's review comment. Created 5 years, 8 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: ui/base/ios/cru_context_menu_controller_unittest.mm
diff --git a/ui/base/ios/cru_context_menu_controller_unittest.mm b/ui/base/ios/cru_context_menu_controller_unittest.mm
index d7b4534129b144ff5445d274a2ce96e9ca82f649..31863d78648d010ef36bc05b6d90cbeb051cf2c7 100644
--- a/ui/base/ios/cru_context_menu_controller_unittest.mm
+++ b/ui/base/ios/cru_context_menu_controller_unittest.mm
@@ -41,13 +41,6 @@ TEST_F(ContextMenuControllerTest, OneEntry) {
[_menuController showWithHolder:holder atPoint:CGPointZero inView:_window];
EXPECT_TRUE([_menuController isVisible]);
-
- UIActionSheet* sheet = _menuController.get().sheet;
- [sheet dismissWithClickedButtonIndex:0 animated:NO];
- [_menuController actionSheet:sheet didDismissWithButtonIndex:0];
-
- EXPECT_TRUE(clicked);
- EXPECT_FALSE([_menuController isVisible]);
}
TEST_F(ContextMenuControllerTest, ShouldDismissImmediately) {
« ui/base/ios/cru_context_menu_controller.mm ('K') | « ui/base/ios/cru_context_menu_controller.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698