Don't allow profile deletion in Metro mode.

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 68c75870c2c0ca005a756ec3969a37775e537350..b7e1fcb09cccff6d5f813cfbd259ee65a8ff150d 100644
--- a/chrome/browser/ui/webui/options/browser_options_handler.cc
+++ b/chrome/browser/ui/webui/options/browser_options_handler.cc
@@ -22,6 +22,7 @@
 #include "base/strings/utf_string_conversions.h"
 #include "base/value_conversions.h"
 #include "base/values.h"
+#include "base/win/metro.h"
 #include "chrome/browser/auto_launch_trial.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ -608,6 +609,9 @@ void BrowserOptionsHandler::GetLocalizedValues(base::DictionaryValue* values) {
   if (ShouldShowMultiProfilesUserList())
     values->Set("profilesInfo", GetProfilesInfoList().release());
 
+  // Profile deletion is not allowed in Metro mode.
+  values->SetBoolean("allowProfileDeletion", !base::win::IsMetroProcess());

[Evan Stade, 2015/01/26 17:55:26]

don't you need some ifdefs

[noms (inactive), 2015/01/26 23:08:14]

Gah, sorry about that. Dropped that ball. Done.

[tapted, 2015/01/26 23:36:07]

I'm pretty sure base::win::IsMetroProcess() only returns true for the viewer
process, not the browser process. aura::RemoteWindowTreeHostWin::IsValid() is
probably the right way to this. Or ash::Shell::HasInstance(), since you can't
delete profiles this way in ChromeOS either.

ash::Shell::HasInstance() has a nice ring to it, since the comment above could
say something like. ~"If the ash desktop is instantiated (e.g. Windows Metro
Mode), the user might delete the Profile that it's using, which is not
supported."

(note the settings webui can still show in Desktop mode while the Chrome browser
is in Metro mode, so we wouldn't want that allowing profile deletion either).

There are still a disturbing number of callers of IsMetroProcess in c/b/ui
though. It's likely they were missed in the great win-aura purges. Probably
there should be a DEPS rule for metro.h to prevent non-viewer-process code using
it :/

[noms (inactive), 2015/01/28 02:20:13]

Sigh. It had such a promising name. I changed it to the one that actually checks
the right thing. Thanks!

+
   values->SetBoolean("profileIsGuest",
                      Profile::FromWebUI(web_ui())->IsOffTheRecord());