Linux: Detect Unity as a desktop environment.

chrome/browser/ui/gtk/gtk_theme_service.cc

Index: chrome/browser/ui/gtk/gtk_theme_service.cc
===================================================================
--- chrome/browser/ui/gtk/gtk_theme_service.cc	(revision 146030)
+++ chrome/browser/ui/gtk/gtk_theme_service.cc	(working copy)
@@ -617,11 +617,17 @@
 
   switch (base::nix::GetDesktopEnvironment(env.get())) {
     case base::nix::DESKTOP_ENVIRONMENT_GNOME:
+    case base::nix::DESKTOP_ENVIRONMENT_UNITY:
     case base::nix::DESKTOP_ENVIRONMENT_XFCE:
       return true;
-    default:
+    case base::nix::DESKTOP_ENVIRONMENT_KDE3:
+    case base::nix::DESKTOP_ENVIRONMENT_KDE4:
+    case base::nix::DESKTOP_ENVIRONMENT_OTHER:
       return false;
   }
+  // g++ can't figure out the switch statement has all cases covered.

[Mike Mammarella, 2012/07/12 01:15:55]

Technically, it can, it's just not sure that somebody won't cast a random int to
the enum type and then have it fall out the end of the switch where there's no
return statement.

[Lei Zhang, 2012/07/12 02:27:02]

Sure, I changed the comment.

+  NOTREACHED();
+  return false;
 }
 
 void GtkThemeService::ClearAllThemeData() {