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

Unified Diff: trunk/src/chrome/browser/ui/cocoa/profiles/profile_chooser_controller_unittest.mm

Issue 464673002: Revert 288817 "[Mac, Win] Only show the user manager tutorial if..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 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: trunk/src/chrome/browser/ui/cocoa/profiles/profile_chooser_controller_unittest.mm
===================================================================
--- trunk/src/chrome/browser/ui/cocoa/profiles/profile_chooser_controller_unittest.mm (revision 288827)
+++ trunk/src/chrome/browser/ui/cocoa/profiles/profile_chooser_controller_unittest.mm (working copy)
@@ -101,9 +101,8 @@
subviews = [[subviews objectAtIndex:0] subviews];
// Three profiles means we should have one active card, one separator and
- // one option buttons view. We also have an update promo for the new avatar
- // menu.
- ASSERT_EQ(4U, [subviews count]);
+ // one option buttons view.
+ ASSERT_EQ(3U, [subviews count]);
// There should be two buttons and a separator in the option buttons view.
NSArray* buttonSubviews = [[subviews objectAtIndex:0] subviews];
@@ -166,9 +165,8 @@
// Three profiles means we should have one active card and a
// fast user switcher which has two "other" profiles and 2 separators, and
- // an option buttons view with its separator. We also have a promo for
- // the new avatar menu.
- ASSERT_EQ(8U, [subviews count]);
+ // an option buttons view with its separator.
+ ASSERT_EQ(7U, [subviews count]);
// There should be two buttons and a separator in the option buttons view.
// These buttons are tested in InitialLayoutWithNewMenu.
@@ -239,9 +237,8 @@
@"Test 1",
@"Test 2" };
// There are four "other" profiles, each with a button and a separator, an
- // active profile card, and an option buttons view with a separator. We
- // also have an update promo for the new avatar menu.
- ASSERT_EQ(12U, [subviews count]);
+ // active profile card, and an option buttons view with a separator.
+ ASSERT_EQ(11U, [subviews count]);
// There should be four "other profiles" items, sorted alphabetically. The
// "other profiles" start at index 2 (after the option buttons view and its
// separator), and each have a separator. We need to iterate through the

Powered by Google App Engine
This is Rietveld 408576698