Do not display lock for hosted domains

chrome/browser/profiles/profile_manager.cc

Index: chrome/browser/profiles/profile_manager.cc
diff --git a/chrome/browser/profiles/profile_manager.cc b/chrome/browser/profiles/profile_manager.cc
index 20cb9d5d2b6f0975a101f129eff1cbe358c3098b..9cdd31cce69d34652ab745c7dc3f06ae9b4143bf 100644
--- a/chrome/browser/profiles/profile_manager.cc
+++ b/chrome/browser/profiles/profile_manager.cc
@@ -794,6 +794,12 @@ void ProfileManager::InitProfileUserPrefs(Profile* profile) {
     profile->GetPrefs()->SetString(prefs::kSupervisedUserId,
                                    supervised_user_id);
   }
+
+#if defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_LINUX)

[msw, 2014/09/19 21:47:40]

This will include Chrome OS, which defines both OS_LINUX and OS_CHROMEOS, is
that your intent? Why limit platforms in this way at all? Shouldn't checking
IsNewProfileManagement suffice here (or is that feature available on some
platform where this particular bit of functionality is not)?

[Mike Lerman, 2014/09/21 00:19:35]

That is not the intent - I want to target Linux Desktop but not ChromeOS. Is
there a better #ifdef for that? Checking IsNewProfileManagement() will
(intentionally) return true on ChromeOS (and Android), but should not be
considered a key for this particular functionality.

[Mike Lerman, 2014/09/22 13:18:56]

I'll flip the OS defs around so I check iOS, Android, CrOS.

[msw, 2014/09/22 19:58:09]

Acknowledged.

+  // If the lock enabled algorithm changed, update this profile's lock status.
+  if (switches::IsNewProfileManagement())
+    profiles::UpdateIsProfileLockEnabledIfNeeded(profile);
+#endif
 }
 
 void ProfileManager::RegisterTestingProfile(Profile* profile,