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

Unified Diff: trunk/src/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.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.mm
===================================================================
--- trunk/src/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm (revision 288827)
+++ trunk/src/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm (working copy)
@@ -818,12 +818,12 @@
}
- (IBAction)showUserManager:(id)sender {
- chrome::ShowUserManager(browser_->profile()->GetPath());
+ profiles::ShowUserManagerMaybeWithTutorial(browser_->profile());
}
- (IBAction)exitGuest:(id)sender {
DCHECK(browser_->profile()->IsGuestSession());
- chrome::ShowUserManager(base::FilePath());
+ [self showUserManager:sender];
profiles::CloseGuestProfileWindows();
}
@@ -849,6 +849,7 @@
[self initMenuContentsWithView:profiles::BUBBLE_VIEW_MODE_GAIA_SIGNIN];
}
+
- (IBAction)addAccount:(id)sender {
[self initMenuContentsWithView:profiles::BUBBLE_VIEW_MODE_GAIA_ADD_ACCOUNT];
[self postActionPerformed:ProfileMetrics::PROFILE_DESKTOP_MENU_ADD_ACCT];
@@ -918,6 +919,10 @@
[self initMenuContentsWithView:profiles::BUBBLE_VIEW_MODE_PROFILE_CHOOSER];
}
+- (IBAction)addPerson:(id)sender {
+ profiles::ShowUserManagerMaybeWithTutorial(browser_->profile());
+}
+
- (IBAction)disconnectProfile:(id)sender {
chrome::ShowSettings(browser_);
}
@@ -1166,6 +1171,11 @@
Profile* profile = browser_->profile();
const AvatarMenu::Item& avatarItem =
avatarMenu_->GetItemAt(avatarMenu_->GetActiveProfileIndex());
+ if (!avatarItem.signed_in) {
+ profile->GetPrefs()->SetInteger(
+ prefs::kProfileAvatarTutorialShown, kUpgradeWelcomeTutorialShowMax + 1);
+ return nil;
+ }
const int showCount = profile->GetPrefs()->GetInteger(
prefs::kProfileAvatarTutorialShown);
@@ -1184,9 +1194,8 @@
IDS_PROFILES_WELCOME_UPGRADE_TUTORIAL_TITLE, avatarItem.name);
NSString* contentMessage = l10n_util::GetNSString(
IDS_PROFILES_WELCOME_UPGRADE_TUTORIAL_CONTENT_TEXT);
- // For local profiles, the "Not you" link doesn't make sense.
- NSString* linkMessage = avatarItem.signed_in ?
- l10n_util::GetNSStringF(IDS_PROFILES_NOT_YOU, avatarItem.name) : nil;
+ NSString* linkMessage = l10n_util::GetNSStringF(
+ IDS_PROFILES_NOT_YOU, avatarItem.name);
NSString* buttonMessage = l10n_util::GetNSString(
IDS_PROFILES_TUTORIAL_WHATS_NEW_BUTTON);
return [self tutorialViewWithMode:profiles::TUTORIAL_MODE_WELCOME_UPGRADE
@@ -1230,27 +1239,24 @@
buttonSize.height += 2 * kTopBottomTextPadding;
[tutorialOkButton setFrameSize:buttonSize];
[tutorialOkButton setAlignment:NSCenterTextAlignment];
-
[tutorialOkButton setFrameOrigin:NSMakePoint(
kFixedMenuWidth - NSWidth([tutorialOkButton frame]) - kHorizontalSpacing,
yOffset)];
[container addSubview:tutorialOkButton];
- if (linkMessage) {
- NSButton* learnMoreLink =
- [self linkButtonWithTitle:linkMessage
- frameOrigin:NSZeroPoint
- action:linkAction];
- [[learnMoreLink cell] setTextColor:[NSColor whiteColor]];
- CGFloat linkYOffset = yOffset + (NSHeight([tutorialOkButton frame]) -
- NSHeight([learnMoreLink frame])) / 2;
- [learnMoreLink setFrameOrigin:NSMakePoint(kHorizontalSpacing, linkYOffset)];
- [container addSubview:learnMoreLink];
- yOffset = std::max(NSMaxY([learnMoreLink frame]),
- NSMaxY([tutorialOkButton frame])) + kVerticalSpacing;
- } else {
- yOffset = NSMaxY([tutorialOkButton frame]) + kVerticalSpacing;
- }
+ NSButton* learnMoreLink =
+ [self linkButtonWithTitle:linkMessage
+ frameOrigin:NSZeroPoint
+ action:linkAction];
+ [[learnMoreLink cell] setTextColor:[NSColor whiteColor]];
+ CGFloat linkYOffset = yOffset + (NSHeight([tutorialOkButton frame]) -
+ NSHeight([learnMoreLink frame])) / 2;
+ [learnMoreLink setFrameOrigin:NSMakePoint(kHorizontalSpacing, linkYOffset)];
+ [container addSubview:learnMoreLink];
+
+ yOffset = std::max(NSMaxY([learnMoreLink frame]),
+ NSMaxY([tutorialOkButton frame])) + kVerticalSpacing;
+
// Adds body content.
NSTextField* contentLabel = BuildLabel(
contentMessage,
@@ -1764,6 +1770,7 @@
return container.autorelease();
}
+
- (NSView*)buildSwitchUserView {
base::scoped_nsobject<NSView> container(
[[NSView alloc] initWithFrame:NSZeroRect]);
@@ -1799,7 +1806,7 @@
text:l10n_util::GetNSString(
IDS_PROFILES_ADD_PERSON_BUTTON)
imageResourceId:IDR_ICON_PROFILES_MENU_AVATAR
- action:@selector(showUserManager:)];
+ action:@selector(addPerson:)];
[container addSubview:addPersonButton];
yOffset = NSMaxY([addPersonButton frame]);

Powered by Google App Engine
This is Rietveld 408576698