Show only the settings pages for new managed users

chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc

Index: chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
diff --git a/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc b/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
index bde0c6c5fe3a8afd0fd613b790f72ab3a04b3ad8..44fd1438c4a318fddffa6045d8a1ba4bd59a4bda 100644
--- a/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
+++ b/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
@@ -133,6 +133,10 @@ bool SyncPromoUI::ShouldShowSyncPromo(Profile* profile) {
   if (profile->IsOffTheRecord())
     return false;
 
+  // Don't show for managed users.

[Dan Beam, 2013/04/03 23:39:20]

nit: managed profiles

[Sergiu, 2013/04/04 09:02:55]

Done.

+  if (profile->GetPrefs()->GetBoolean(prefs::kProfileIsManaged))

[Dan Beam, 2013/04/03 23:39:20]

do you want to disable the promo based on whether the *profile* is managed or
whether *sync* is managed?  if it's sync, you should probably use

  if (!profile->IsSyncAccessible())

instead

[Dan Beam, 2013/04/04 00:01:45]

er, if (profile->IsSyncAccessible()) I mean

[Dan Beam, 2013/04/04 01:12:59]

... and I'm going to go drink more coffee

[Sergiu, 2013/04/04 09:02:55]

It is for managed users (see the referenced bug for our definition of a managed
user).

+    return false;
+
   // Display the signin promo if the user is not signed in.
   SigninManager* signin = SigninManagerFactory::GetForProfile(
       profile->GetOriginalProfile());