Add the calculated WMClass to generated .desktop files.

chrome/browser/ui/gtk/browser_window_gtk.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/gtk/browser_window_gtk.h"
 
 #include <gdk/gdkkeysyms.h>
 
 #include <string>
 
@@ skipping 53 matching lines @@
 #include "chrome/browser/ui/gtk/status_bubble_gtk.h"
 #include "chrome/browser/ui/gtk/tab_contents_container_gtk.h"
 #include "chrome/browser/ui/gtk/tabs/tab_strip_gtk.h"
 #include "chrome/browser/ui/gtk/task_manager_gtk.h"
 #include "chrome/browser/ui/gtk/theme_install_bubble_view_gtk.h"
 #include "chrome/browser/ui/gtk/update_recommended_dialog.h"
 #include "chrome/browser/ui/omnibox/location_bar.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/browser/ui/webui/bug_report_ui.h"
 #include "chrome/browser/ui/window_sizer.h"
+#include "chrome/browser/web_applications/web_app.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "content/browser/renderer_host/render_widget_host_view.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/browser/tab_contents/tab_contents_view.h"
 #include "content/common/native_web_keyboard_event.h"
 #include "content/common/notification_service.h"
 #include "grit/app_resources.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
@@ skipping 203 matching lines @@
   g_object_set_qdata(G_OBJECT(window_), GetBrowserWindowQuarkKey(), this);
   gtk_widget_add_events(GTK_WIDGET(window_), GDK_BUTTON_PRESS_MASK |
                                              GDK_POINTER_MOTION_MASK);
 
   // Add this window to its own unique window group to allow for
   // window-to-parent modality.
   gtk_window_group_add_window(gtk_window_group_new(), window_);
   g_object_unref(gtk_window_get_group(window_));
 
   if (browser_->type() & Browser::TYPE_APP) {
-    std::string wmclassname = browser_->app_name();
-    if (wmclassname != DevToolsWindow::kDevToolsApp) {
-      file_util::ReplaceIllegalCharactersInPath(&wmclassname, '_');
-      TrimString(wmclassname, "_", &wmclassname);
-      gtk_window_set_wmclass(window_, wmclassname.c_str(),
-                             wmclassname.c_str());
+    std::string app_name = browser_->app_name();
+    if (app_name != DevToolsWindow::kDevToolsApp) {
+      std::string wmclassname = web_app::GetWMClassFromAppName(app_name);
+      gtk_window_set_wmclass(window_,
+                             wmclassname.c_str(),
+                             gdk_get_program_class());

[Marco Trevisan (Treviño), 2011/04/21 15:53:42]

This is wrong: you must set the wmclassname as third parameter of
gtk_window_set_wmclass for it to be recognized by window managers (which are
using libraries like libwnck)!

Best implementation would be:
gtk_window_set_wmclass(window_, window_->wmclass_name, wmclassname.c_str());

And also you can add too:
gtk_window_set_role(window_, wmclassname.c_str());

     }
   }
 
   // For popups, we initialize widgets then set the window geometry, because
   // popups need the widgets inited before they can set the window size
   // properly. For other windows, we set the geometry first to prevent resize
   // flicker.
   if (browser_->type() & Browser::TYPE_POPUP) {
     InitWidgets();
     SetGeometryHints();
@@ skipping 1978 matching lines @@
   // special-case the ones where the custom frame should be used.  These names
   // are taken from the WMs' source code.
   return (wm_name == "Blackbox" ||
           wm_name == "compiz" ||
           wm_name == "e16" ||  // Enlightenment DR16
           wm_name == "Metacity" ||
           wm_name == "Mutter" ||
           wm_name == "Openbox" ||
           wm_name == "Xfwm4");
 }