[Win] The new profiles UI: now at an immersive mode near you

chrome/browser/ui/webui/options/browser_options_handler.cc

Index: chrome/browser/ui/webui/options/browser_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/browser_options_handler.cc b/chrome/browser/ui/webui/options/browser_options_handler.cc
index b8cf8c189fb42dcd109f0ea946ab1fb7d71f5698..4d62d79a383f8dc9c1e7f49f13ba4bbdee809aa9 100644
--- a/chrome/browser/ui/webui/options/browser_options_handler.cc
+++ b/chrome/browser/ui/webui/options/browser_options_handler.cc
@@ -1039,8 +1039,11 @@ bool BrowserOptionsHandler::ShouldShowMultiProfilesUserList() {
   // On Chrome OS we use different UI for multi-profiles.
   return false;
 #else
-  if (helper::GetDesktopType(web_ui()) != chrome::HOST_DESKTOP_TYPE_NATIVE)
+  chrome::HostDesktopType desktop_type = helper::GetDesktopType(web_ui());
+  if (desktop_type != chrome::HOST_DESKTOP_TYPE_NATIVE &&

[msw, 2014/10/21 19:54:38]

What? I don't think there is any other desktop type besides native or ash...
afaict, this if statement will never evaluate to true.

[noms (inactive), 2014/10/21 20:55:18]

Oh, yeah, you're totally right. I misread that enum as having 4 values, but
that's a lie.

+      desktop_type != chrome::HOST_DESKTOP_TYPE_ASH) {
     return false;
+  }
   Profile* profile = Profile::FromWebUI(web_ui());
   if (profile->IsGuestSession())
     return false;