Nicely handle OnBeforeUnloads with guest and lock mode.

chrome/browser/profiles/profile_window.cc

Index: chrome/browser/profiles/profile_window.cc
diff --git a/chrome/browser/profiles/profile_window.cc b/chrome/browser/profiles/profile_window.cc
index 10b48e19cd03d7fa56cab89d3d442126198abc31..811b2ef19840fd43906b382e70578fef89326f93 100644
--- a/chrome/browser/profiles/profile_window.cc
+++ b/chrome/browser/profiles/profile_window.cc
@@ -259,25 +259,38 @@ void CreateAndSwitchToNewProfile(chrome::HostDesktopType desktop_type,
   ProfileMetrics::LogProfileAddNewUser(metric);
 }
 
+void GuestBrowserCloseSuccess(const base::FilePath& profile_path) {
+  chrome::ShowUserManager(profile_path);
+}
+
 void CloseGuestProfileWindows() {
   ProfileManager* profile_manager = g_browser_process->profile_manager();
   Profile* profile = profile_manager->GetProfileByPath(
       ProfileManager::GetGuestProfilePath());
 
   if (profile) {
-    BrowserList::CloseAllBrowsersWithProfile(profile);
+    BrowserList::CloseAllBrowsersWithProfile(
+        profile,
+        base::Bind(&GuestBrowserCloseSuccess));

[noms (inactive), 2014/09/03 14:25:08]

nit: i think the base::Bind can go on the same line as profile

[Mike Lerman, 2014/09/03 15:28:34]

Done.

   }
 }
 
+void LockBrowserCloseSuccess(const base::FilePath& profile_path) {
+  ProfileInfoCache* cache =
+      &g_browser_process->profile_manager()->GetProfileInfoCache();
+
+  cache->SetProfileSigninRequiredAtIndex(
+      cache->GetIndexOfProfileWithPath(profile_path),
+      true);

[Peter Kasting, 2014/09/02 19:40:34]

Nit: Can place on end of previous line if you want

[Mike Lerman, 2014/09/03 15:28:34]

Done.

+  chrome::ShowUserManager(profile_path);
+}
+
 void LockProfile(Profile* profile) {
   DCHECK(profile);
-  ProfileInfoCache& cache =
-      g_browser_process->profile_manager()->GetProfileInfoCache();
-
-  size_t index = cache.GetIndexOfProfileWithPath(profile->GetPath());
-  cache.SetProfileSigninRequiredAtIndex(index, true);
-  chrome::ShowUserManager(profile->GetPath());
-  BrowserList::CloseAllBrowsersWithProfile(profile);
+  if (profile) {
+    BrowserList::CloseAllBrowsersWithProfile(profile,

[Peter Kasting, 2014/09/02 19:40:34]

Nit: Place |profile| on next line; you may leave the 2nd arg on that same line
if you want

(If you end up with multiple lines of args, they need to start at the same
position.)

[Mike Lerman, 2014/09/03 15:28:34]

Done.

+        base::Bind(&LockBrowserCloseSuccess));
+  }
 }
 
 void CreateGuestProfileForUserManager(